[PATCH] D79294: [InstSimplify] Remove known bits constant folding

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun May 3 04:15:09 PDT 2020


nikic updated this revision to Diff 261690.
nikic added a comment.
Herald added a subscriber: ormris.

I have now duplicated all the relevant tests into InstCombine and added comments that these no longer fold as part of InstSimplify. The only test I've dropped outright is the assume.ll one, as I don't think it makes sense if we don't perform top-level KnownBits folding. I've left the remaining ones alone, but could drop them if desired.


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

https://reviews.llvm.org/D79294

Files:
  lib/Analysis/InstructionSimplify.cpp
  test/Analysis/ValueTracking/knownzero-shift.ll
  test/Transforms/GVN/PRE/volatile.ll
  test/Transforms/InstSimplify/assume.ll
  test/Transforms/InstSimplify/call.ll
  test/Transforms/InstSimplify/or.ll
  test/Transforms/InstSimplify/shift-knownbits.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D79294.261690.patch
Type: text/x-patch
Size: 13062 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200503/1d8dec79/attachment.bin>


More information about the llvm-commits mailing list