Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Windows changes for Makefile #236

Closed
wants to merge 4 commits into from
Closed

Windows changes for Makefile #236

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
ad77151
Windows changes for Makefile
rosslwheeler Apr 24, 2024
04085a8
Updating with @Ricardicus comments
rosslwheeler Apr 24, 2024
d8fb206
Quick comment change
rosslwheeler Apr 24, 2024
1d1ea11
Merge branch 'karpathy:master' into makefile_windows_update
rosslwheeler Apr 25, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ else
INCLUDES :=
NVCC_FLAGS += -I"dev"
WIN_CUDA_RENAME = rename [email protected] $@
OUTPUT_FILE = /link /OUT:$@
OUTPUT_FILE = /link /OUT:$@
endif

# Check if OpenMP is available
Expand Down Expand Up @@ -85,8 +85,8 @@ else
endif
endif

# DEFAULT means these targets will always be executed
.DEFAULT: all
# Force the rebuild of targets - CI relies on this.
rosslwheeler marked this conversation as resolved.
Show resolved Hide resolved
.PHONY: all train_gpt2 test_gpt2 train_gpt2cu test_gpt2cu train_gpt2fp32cu test_gpt2fp32cu

# Add targets
TARGETS = train_gpt2 test_gpt2
Expand Down
Loading