[PATCH] D18701: [ARM] Adding IEEE-754 SIMD detection to loop vectorizer

[Renato Golin via llvm-commits]

rengolin updated this revision to Diff 52780.
rengolin added a comment.

Update from reviews:

- Adding a FIXME comment to the vectorizer's check, make clear how big the hammer is
- Being even more conservative, using all possible types of instructions (now including shifts and function calls, even those that will be builtins)
- Changing the CHECK lines to use the legality message, not the final vectorization one, which can change over time and is irrelevant to the patch
- Adding two new cases for fabs() with and without fast-math

Checked that the behaviour is consistent with GCC.


http://reviews.llvm.org/D18701

Files:
  include/llvm/Analysis/TargetTransformInfo.h
  include/llvm/Analysis/TargetTransformInfoImpl.h
  lib/Analysis/TargetTransformInfo.cpp
  lib/Target/ARM/ARMTargetTransformInfo.h
  lib/Transforms/Vectorize/LoopVectorize.cpp
  test/Transforms/LoopVectorize/ARM/arm-ieee-vectorize.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D18701.52780.patch
Type: text/x-patch
Size: 22796 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160406/114aa26b/attachment.bin>