[PATCH] D86863: [EarlyCSE] Verify hash code in regression tests

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 31 02:10:05 PDT 2020


lebedev.ri accepted this revision.
lebedev.ri added a comment.
This revision is now accepted and ready to land.

Test changes are certainly LGTM, may be best to commit them separately.

For the record, i don't know just how slower this makes EarlyCSE in `EXPENSIVE_CHECKS` build, but i can't imagine having lurking nondeterminism with an easy way to catch it is better.
@spatel thoughts on the switch?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D86863/new/

https://reviews.llvm.org/D86863



More information about the llvm-commits mailing list