[PATCH] D19506: LoadStoreVectorizer: Skip optnone functions

[Matt Arsenault via llvm-commits]

arsenm updated this revision to Diff 56812.
arsenm added a comment.

Use skipFunction


http://reviews.llvm.org/D19506

Files:
  lib/Transforms/Vectorize/LoadStoreVectorizer.cpp
  test/Transforms/LoadStoreVectorizer/AMDGPU/optnone.ll

Index: test/Transforms/LoadStoreVectorizer/AMDGPU/optnone.ll
===================================================================
--- /dev/null
+++ test/Transforms/LoadStoreVectorizer/AMDGPU/optnone.ll
@@ -0,0 +1,12 @@
+; RUN: opt -mtriple=amdgcn-amd-amdhsa -load-store-vectorizer -S -o - %s | FileCheck %s
+
+; CHECK-LABEL: @optnone(
+; CHECK: store i32
+; CHECK: store i32
+define void @optnone(i32 addrspace(1)* %out) noinline optnone {
+  %out.gep.1 = getelementptr i32, i32 addrspace(1)* %out, i32 1
+
+  store i32 123, i32 addrspace(1)* %out.gep.1
+  store i32 456, i32 addrspace(1)* %out
+  ret void
+}
Index: lib/Transforms/Vectorize/LoadStoreVectorizer.cpp
===================================================================
--- lib/Transforms/Vectorize/LoadStoreVectorizer.cpp
+++ lib/Transforms/Vectorize/LoadStoreVectorizer.cpp
@@ -155,7 +155,7 @@
     = getAnalysis<TargetTransformInfoWrapperPass>().getTTI(F);
 
   // Don't vectorize when the attribute NoImplicitFloat is used.
-  if (F.hasFnAttribute(Attribute::NoImplicitFloat))
+  if (F.hasFnAttribute(Attribute::NoImplicitFloat) || skipFunction(F))
     return false;
 
   Vectorizer V(F, AA, DT, SE, TTI);


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D19506.56812.patch
Type: text/x-patch
Size: 1174 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160510/376658e9/attachment.bin>