[PATCH] D93015: Fold comparison of __builtin_object_size expression with -1 for non-const size

Siddhesh Poyarekar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 18 11:19:34 PST 2020


siddhesh updated this revision to Diff 312857.
siddhesh added a comment.

Updated patch.  I'm not entirely sure if I 'fixed' the failing test case (limit-max-iterations.ll) correctly.  Earlier it would reduce in two instcombine iterations but with this assumption, it reduces to its minimal for (i.e. with just the llvm.assume and ret) in just one iteration.


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

https://reviews.llvm.org/D93015

Files:
  llvm/lib/Analysis/MemoryBuiltins.cpp
  llvm/test/Transforms/InstCombine/builtin-dynamic-object-size.ll
  llvm/test/Transforms/InstCombine/limit-max-iterations.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D93015.312857.patch
Type: text/x-patch
Size: 4243 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201218/4501d9d8/attachment.bin>


More information about the llvm-commits mailing list