Skip to content

Conversation

khaledhosny
Copy link
Collaborator

Cuts a few minutes here and there, should make things a bit faster.

@khaledhosny khaledhosny merged commit d2b20ba into harfbuzz:master Oct 24, 2018
@khaledhosny khaledhosny deleted the appveyor-speedup branch October 24, 2018 15:27
@behdad
Copy link
Member

behdad commented Oct 24, 2018

Thanks! Is -j okay or will worse things? Fine with me.

@khaledhosny
Copy link
Collaborator Author

I’m not sure either, builds seem to vary by several minutes on each job between runs, so it is hard to measure if it is making things better or worse.

@behdad
Copy link
Member

behdad commented Oct 24, 2018

Maybe just -j4 or something. Not sure how easy it is to overwhelm the Windows scheduler...

@khaledhosny
Copy link
Collaborator Author

This thread suggests that there are two cores in the VM, so I guess -j2 then.

@behdad
Copy link
Member

behdad commented Oct 24, 2018

Even with one core, -j2 runs faster than single-process. That's because IO and CPU tasks will be interleaved, instead of stalled for each other.

@behdad
Copy link
Member

behdad commented Oct 24, 2018

-j3 maybe then.

@khaledhosny
Copy link
Collaborator Author

Trying that now.

@khaledhosny
Copy link
Collaborator Author

-j3 seems to be three minutes faster per each CI job than -j2.

khaledhosny added a commit that referenced this pull request Oct 25, 2018
ebraminio pushed a commit to ebraminio/harfbuzz that referenced this pull request Nov 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants