[PATCH] D27618: Failure to vectorize __builtin_sqrt/__builtin_sqrtf

Andrew V. Tischenko via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 19 03:46:02 PST 2016


avt77 updated this revision to Diff 81934.
avt77 added a comment.
Herald added a subscriber: mzolotukhin.

I restored the check if (ICS->hasNoNaNs()) but for non-vector operations only because vector instructions work correctly with invalid input values. To make it possible I changed the signature of llvm::getIntrinsicForCallSite. Now it knows the required intrinsic target.


https://reviews.llvm.org/D27618

Files:
  include/llvm/Analysis/ValueTracking.h
  lib/Analysis/ValueTracking.cpp
  lib/Analysis/VectorUtils.cpp
  test/CodeGen/X86/vector-sqrt.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D27618.81934.patch
Type: text/x-patch
Size: 6030 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161219/f455af62/attachment.bin>


More information about the llvm-commits mailing list