[PATCH] D11753: [X86] Add support for _MM_ALIGN16

[Michael Kuperstein]

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