echristo accepted this revision. echristo added a comment. This revision is now accepted and ready to land.
Couple of inline comments, otherwise LGTM. -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 - \ ---------------- Any chance of putting thi with the rest of the Intrin.h checks? ================ 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 ---------------- Shouldn't need -Oz. http://reviews.llvm.org/D11753 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
