[PATCH] Teach scalarrepl about address spaces
Quentin Colombet
qcolombet at apple.com
Wed Oct 30 12:51:48 PDT 2013
Hi Matt,
LGTM with one minor comment:
One of your test is missing CHECK lines or should be removed.
Thanks,
-Quentin
================
Comment at: test/Transforms/ScalarRepl/union-pointer.ll:2
@@ -1,5 +1,3 @@
; PR892
-; RUN: opt < %s -scalarrepl -S | \
-; RUN: not grep alloca
-; RUN: opt < %s -scalarrepl -S | grep "ret i8"
+; RUN: opt < %s -scalarrepl -S | FileCheck %s
----------------
Thanks for this clean up.
================
Comment at: test/Transforms/ScalarRepl/union-pointer.ll:33
@@ +32,3 @@
+
+define i8 addrspace(1)* @test_as1_array(i16 addrspace(1)* %x) {
+ %as_ptr_array = alloca [4 x i16 addrspace(1)*]
----------------
Is this missing some CHECK lines or should it be removed?
http://llvm-reviews.chandlerc.com/D1779
More information about the llvm-commits
mailing list