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

Travis: Enable cpu-specific optimizations. #17

Merged
merged 1 commit into from
Aug 29, 2018

Conversation

c0gent
Copy link
Contributor

@c0gent c0gent commented Aug 29, 2018

I'm not sure how much if any speedup this gives but there are no drawbacks I'm aware of (unless lto=true, which we won't be using).

@vkomenda
Copy link
Contributor

vkomenda commented Aug 29, 2018

There was an issue related to that a couple of weeks ago: rust-lang/rust#53322.

...As Marc already pointed out before.

@c0gent
Copy link
Contributor Author

c0gent commented Aug 29, 2018

Looks fixed already (usually nightly stuff is fixed pretty quickly).

@afck
Copy link
Collaborator

afck commented Aug 29, 2018

Looks good! The older commit has a different hash from the one in master, though, so it should probably first be rebased manually.

@c0gent
Copy link
Contributor Author

c0gent commented Aug 29, 2018

Yeah those go away if you Rebase/Squash and merge but... done.

@afck afck merged commit 43799fc into poanetwork:master Aug 29, 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