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

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Nov 19 10:34:58 PST 2017


spatel updated this revision to Diff 123504.
spatel added a reviewer: craig.topper.
spatel added a comment.

Patch updated:
Add a specialized TTI hook as suggested. So now this becomes a functional change only for x86 which overrides the default hook.


https://reviews.llvm.org/D28314

Files:
  include/llvm/Analysis/TargetTransformInfo.h
  include/llvm/Analysis/TargetTransformInfoImpl.h
  include/llvm/CodeGen/BasicTTIImpl.h
  lib/Analysis/TargetTransformInfo.cpp
  lib/Target/X86/X86TargetTransformInfo.cpp
  lib/Target/X86/X86TargetTransformInfo.h
  lib/Transforms/Scalar/PartiallyInlineLibCalls.cpp
  test/CodeGen/X86/sqrt-partial.ll
  test/Transforms/PartiallyInlineLibCalls/X86/good-prototype.ll

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


More information about the llvm-commits mailing list