[PATCH] D28938: Implement P0513R0 - "Poisoning the Hash"

2017-01-20 Thread Eric Fiselier via Phabricator via cfe-commits
EricWF accepted this revision. EricWF added a comment. This revision is now accepted and ready to land. I feel comfortable with this in its current state. https://reviews.llvm.org/D28938 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D28938: Implement P0513R0 - "Poisoning the Hash"

2017-01-20 Thread Eric Fiselier via Phabricator via cfe-commits
EricWF updated this revision to Diff 85172. EricWF added a comment. - Fix bitset test that was overflowing the stack. - Add tests that basic_string_view hashes to the same value as an equivalent basic_string. https://reviews.llvm.org/D28938 Files: include/__functional_base

[PATCH] D28938: Implement P0513R0 - "Poisoning the Hash"

2017-01-19 Thread Eric Fiselier via Phabricator via cfe-commits
EricWF created this revision. Exactly what the title says. This patch also adds a `std::hash` 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