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

Renato Golin via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 25 09:40:18 PST 2016


rengolin added inline comments.

================
Comment at: lib/Target/ARM/ARMTargetTransformInfo.h:59
@@ +58,3 @@
+
+  // Darwin doesn't care about subnormals, so fast math is allowed everywhere.
+  bool supportsSubnormal() { return !ST->isTargetDarwin(); }
----------------
hfinkel wrote:
> This comment seems misleading. "fast math" normally implies allowing things like reassociation (and lack of NaNs, etc.), and that has nothing to do with subnormals.
Indeed, I'll remove this comment.


http://reviews.llvm.org/D17141





More information about the llvm-commits mailing list