[PATCH] D19506: LoadStoreVectorizer: Skip optnone functions

Matthias Braun via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 30 16:41:39 PDT 2016


MatzeB added a subscriber: MatzeB.

================
Comment at: lib/Transforms/Vectorize/LoadStoreVectorizer.cpp:158
@@ -157,3 +157,3 @@
   // Don't vectorize when the attribute NoImplicitFloat is used.
-  if (F.hasFnAttribute(Attribute::NoImplicitFloat))
+  if (F.hasFnAttribute(Attribute::NoImplicitFloat) || skipFunction(F))
     return false;
----------------
I would recommend to make skipFunction() the first thing in runOnFunction(). Otherwise I am not sure about the effects on opt bisection when the number of calls to skipFunction() depends on the number of NoImplicitFloat attributes in the IR...


http://reviews.llvm.org/D19506





More information about the llvm-commits mailing list