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

[Renato Golin via llvm-commits]

rengolin added inline comments.

================
Comment at: include/llvm/Analysis/TargetTransformInfo.h:373
@@ +372,3 @@
+  /// and SLP vectorization without -ffast-math option.
+  bool isSIMDIEEE754() const;
+
----------------
An alternative to this would be to have an enum:

    enum {
      VFP = 0x1,
      SIMD = 0x2
    } IEEE754Support;

and initialise all targets with 0x3, but ARM with 0x1.

Then to get the value, we do:

    IsSIMDIEEE = getIEEE754Support() & IEEE754Support::SIMD;

But since this patch doesn't need that, I think we should do that later, when it's needed. Changes to the SLP vectorizer will probably do, and I can change it when I get there.


Repository:
  rL LLVM

http://reviews.llvm.org/D17141