This is an archive of the discontinued LLVM Phabricator instance.

[libcxx] [test] Improve portability of hash tests.
ClosedPublic

Authored by STL_MSFT on Jun 14 2016, 2:18 PM.

Details

Summary

Improve portability of hash tests.

These tests were expecting libcxx's identity-hash behavior, which isn't guaranteed by the Standard and isn't provided by MSVC (we currently use FNV-1a for everything). Marking the asserts as libcxx-specific and avoiding unused variable warnings for MSVC makes the tests portable.

Diff Detail

Event Timeline

STL_MSFT updated this revision to Diff 60751.Jun 14 2016, 2:18 PM
STL_MSFT retitled this revision from to [libcxx] [test] Improve portability of hash tests..
STL_MSFT updated this object.
STL_MSFT added reviewers: EricWF, mclow.lists.
STL_MSFT added a subscriber: cfe-commits.
EricWF accepted this revision.Jun 14 2016, 6:49 PM
EricWF edited edge metadata.
This revision is now accepted and ready to land.Jun 14 2016, 6:49 PM
EricWF closed this revision.Jun 14 2016, 6:49 PM

r272744.