chandlerc accepted this revision.
chandlerc added a comment.
This revision is now accepted and ready to land.
Looks good with the two test nits below fixed.
================
Comment at: test/CodeGen/mips-zero-sized-struct.c:8-10
@@ -7,5 +7,5 @@
-// O32: define void @fn28(%struct.T2* noalias sret %agg.result, i8 signext
%arg0)
-// N32: define void @fn28(i8 signext %arg0)
-// N64: define void @fn28(i8 signext %arg0)
+// O32: define void @fn28(%struct.T2* noalias sret{{.*}}, i8 signext{{.*}})
+// N32: define void @fn28(i8 signext{{.*}})
+// N64: define void @fn28(i8 signext{{.*}})
----------------
No need for this part of the change now?
================
Comment at: test/CodeGenCXX/stack-reuse.cpp:1
@@ -1,2 +1,2 @@
-// RUN: %clang -target armv7l-unknown-linux-gnueabihf -S %s -o - -emit-llvm
-O1 -disable-llvm-optzns | FileCheck %s
+// RUN: %clang -cc1 -triple armv7-unknown-linux-gnueabihf %s -o - -emit-llvm
-O1 | FileCheck %s
----------------
Another place to use %clang_cc1 instead of '-cc1'.
http://reviews.llvm.org/D18127
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
