[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