[PATCH] D103016: NewGVN: Relax assertion about number of times a value is seen

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 24 06:23:15 PDT 2021


arsenm created this revision.
arsenm added reviewers: dberlin, davide, fhahn, lebedev.ri, spatel, nikic.
Herald added a subscriber: hiraditya.
arsenm requested review of this revision.
Herald added a subscriber: wdng.
Herald added a project: LLVM.

A value can legitimately appear more than 100 times. This assert in
general seems bogus to me, but raise it to 100 times the number of
blocks in the function which is still arbitrary.

      

Addresses bug 50444.


https://reviews.llvm.org/D103016

Files:
  llvm/lib/Transforms/Scalar/NewGVN.cpp
  llvm/test/Transforms/NewGVN/bug50444.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D103016.347371.patch
Type: text/x-patch
Size: 51193 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210524/e7db3a94/attachment.bin>


More information about the llvm-commits mailing list