-
Notifications
You must be signed in to change notification settings - Fork 13.4k
[do not merge] Test ahash #59592
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
[do not merge] Test ahash #59592
Conversation
(rust_highfive has picked a reviewer for you, use r? to override) |
@bors try |
⌛ Trying commit 1dc107d26711bccaa56e88d5a885fa11e7875c72 with merge 1ffa4f33a75cbc37938ecba338d068137ed91c26... |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
💔 Test failed - checks-travis |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
@bors try |
[do not merge] Test ahash cc @tkaitchuck
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
💔 Test failed - checks-travis |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
Clever renaming AHasher to FxHasher to update everywhere without making the diff huge.
Could that be related to something aHash is doing that Fx is not, like maybe proc macros? |
Nah, it's just a linker error. No idea why it appears though. |
☔ The latest upstream changes (presumably #59695) made this pull request unmergeable. Please resolve the merge conflicts. |
@Zoxc I just came across aHash and was wondering if anything ever came out of testing it in rustc? |
I haven't managed to test it in rustc. |
cc @tkaitchuck