[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