mkuper added a comment.

Thanks, Eric!


================
Comment at: test/CodeGen/ms-mm-align.c:1
@@ +1,2 @@
+// RUN: %clang_cc1 -ffreestanding -fms-extensions -fms-compatibility 
-fms-compatibility-version=17.00 \
+// RUN:         -triple i686--windows -Oz -emit-llvm %s -o - \
----------------
echristo wrote:
> Any chance of putting thi with the rest of the Intrin.h checks?
I've originally wanted to put it in ms-intinsics.c, but it fails for the 
"-triple thumbv7--windows" test (as it should - this is an x86-only macro).
If you have a different test in mind, I'll move it there.

================
Comment at: test/CodeGen/ms-mm-align.c:2
@@ +1,3 @@
+// RUN: %clang_cc1 -ffreestanding -fms-extensions -fms-compatibility 
-fms-compatibility-version=17.00 \
+// RUN:         -triple i686--windows -Oz -emit-llvm %s -o - \
+// RUN:         | FileCheck %s -check-prefix CHECK
----------------
echristo wrote:
> Shouldn't need -Oz.
Copied that from ms-intrinsics.c as well. Will remove.


http://reviews.llvm.org/D11753



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to