[PATCH] D63214: [InstCombine] canonicalize fmin/fmax to LLVM intrinsics minnum/maxnum

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 19 15:42:54 PDT 2019


spatel updated this revision to Diff 205690.
spatel added a comment.

Patch updated:
Based on the discussion in D63294 <https://reviews.llvm.org/D63294>, we can't be too strict about matching libm calls in IR (a mismatched 'f' or 'l' suffix is ok), so I've updated the test that was checking for that case.


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

https://reviews.llvm.org/D63214

Files:
  llvm/lib/Transforms/Utils/SimplifyLibCalls.cpp
  llvm/test/Transforms/InstCombine/double-float-shrink-1.ll
  llvm/test/Transforms/InstCombine/fast-math.ll
  llvm/test/Transforms/InstCombine/float-shrink-compare.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D63214.205690.patch
Type: text/x-patch
Size: 13865 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190619/2c8a228e/attachment.bin>


More information about the llvm-commits mailing list