This is an archive of the discontinued LLVM Phabricator instance.

Implement P0513R0 - "Poisoning the Hash"
ClosedPublic

Authored by EricWF on Jan 19 2017, 10:21 PM.

Details

Summary

Exactly what the title says.

This patch also adds a std::hash<nullptr_t> specialization in C++17, but it was not added by this paper and I can't find the actual paper that adds it.

See http://wg21.link/P0513R0 for more info.

If there are no comments in the next couple of days I'll commit this

Diff Detail

Event Timeline

EricWF created this revision.Jan 19 2017, 10:21 PM
EricWF updated this revision to Diff 85172.Jan 20 2017, 12:13 PM
  • Fix bitset test that was overflowing the stack.
  • Add tests that basic_string_view hashes to the same value as an equivalent basic_string.
EricWF accepted this revision.Jan 20 2017, 4:09 PM

I feel comfortable with this in its current state.

This revision is now accepted and ready to land.Jan 20 2017, 4:09 PM
EricWF closed this revision.Jan 20 2017, 4:13 PM