[PATCH] D28314: Change sqrt partial inlining to depend on sqrt argument rather than result.

Michael Kuperstein via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 4 13:44:52 PST 2017


mkuper created this revision.
mkuper added reviewers: RKSimon, uweigand, sdardis.
mkuper added a subscriber: llvm-commits.

This should do the right thing on X86 and resolve PR31455.
I'm not sure it makes sense for MIPS and SystemZ though, which were the original targets of the pass.


https://reviews.llvm.org/D28314

Files:
  lib/Transforms/Scalar/PartiallyInlineLibCalls.cpp
  test/CodeGen/Mips/optimize-fp-math.ll
  test/CodeGen/SystemZ/fp-sqrt-01.ll
  test/CodeGen/SystemZ/fp-sqrt-02.ll
  test/CodeGen/X86/sqrt-partial.ll
  test/Transforms/PartiallyInlineLibCalls/X86/good-prototype.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D28314.83120.patch
Type: text/x-patch
Size: 5156 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170104/add93822/attachment.bin>


More information about the llvm-commits mailing list