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

Joseph Tremoulet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 31 11:07:02 PDT 2020


JosephTremoulet added a comment.

Yeah, the potential for quadratic explosion is why I didn't hook this flag up to expensive checks when I added it. It occurs to me now that if we want, we could presumably have some cutoff counter so that after inserting 300 instructions or whatever limit is reasonable, we start hashing normally. We'd have to empty the table and repopulate it with the normal hash when we cut over, but that should be doable I think (it's similar to what the table does internally when it grows).


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