[PATCH] D45453: [InstCombine] Enable Add/Sub simplifications with only 'reassoc' FMF

Warren Ristow via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 13 18:12:07 PDT 2018


wristow updated this revision to Diff 142500.
wristow added a comment.

Thanks Sanjay.  Updated the tests to be more careful about when 'nsz' is and isn't needed, along with comments to clarify.  Also simplified some tests where no FMF is needed (irrespective of this patch).


https://reviews.llvm.org/D45453

Files:
  lib/Transforms/InstCombine/InstCombineAddSub.cpp
  test/Transforms/InstCombine/fast-math.ll
  test/Transforms/Reassociate/fast-MissedTree.ll
  test/Transforms/Reassociate/fast-SubReassociate.ll
  test/Transforms/Reassociate/fast-basictest.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D45453.142500.patch
Type: text/x-patch
Size: 29783 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180414/6c9e9991/attachment.bin>


More information about the llvm-commits mailing list