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.

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

Use NMake batch-rules for compilation #9512

Closed
wants to merge 1 commit into from

Conversation

chemodax
Copy link
Contributor

Currently NMake based build invokes cl.exe for every file. This pull request enable NMake batch-mode rules for cl.exe. In this case cl.exe invoked for multiple .c files.

This is significantly improves compilation time. For example in my environment: 40 s → 20 s.

@bagder bagder requested a review from jay September 15, 2022 21:37
@bagder bagder added build Windows Windows-specific labels Sep 15, 2022
@jay jay closed this in 8a7ecb5 Sep 16, 2022
@jay
Copy link
Member

jay commented Sep 16, 2022

This is significantly improves compilation time. For example in my environment: 40 s → 20 s.

I see the same. Thanks

jquepi pushed a commit to jquepi/curl.1.555 that referenced this pull request Oct 24, 2022
- Invoke cl compiler once for each group of .c files.

This is significantly improves compilation time. For example in my
environment: 40 s --> 20 s.

Prior to this change cl was invoked per .c file.

Closes curl/curl#9512
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Windows Windows-specific
Development

Successfully merging this pull request may close these issues.

None yet

3 participants