r194249 - Improve the missing ASM parser test for MS-style assembly

Alp Toker alp at nuanti.com
Thu Nov 7 22:50:48 PST 2013


Author: alp
Date: Fri Nov  8 00:50:48 2013
New Revision: 194249

URL: http://llvm.org/viewvc/llvm-project?rev=194249&view=rev
Log:
Improve the missing ASM parser test for MS-style assembly

It's better to test clang-check rather than the internal c-index-test utility.

Also adds a target so we can remove the XFAILs.

Thanks to Richard Barton for spotting the test failure on ARM.

Test originally from r193685.

Added:
    cfe/trunk/test/Tooling/ms-asm-no-target.cpp
Removed:
    cfe/trunk/test/Index/ms-asm-no-target.cpp

Removed: cfe/trunk/test/Index/ms-asm-no-target.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Index/ms-asm-no-target.cpp?rev=194248&view=auto
==============================================================================
--- cfe/trunk/test/Index/ms-asm-no-target.cpp (original)
+++ cfe/trunk/test/Index/ms-asm-no-target.cpp (removed)
@@ -1,11 +0,0 @@
-// XFAIL: aarch64, arm, mips, hexagon, powerpc, sparc
-// REQUIRES: x86-registered-target
-// RUN: c-index-test -test-load-source all -fasm-blocks -Wno-microsoft %s 2>&1 | FileCheck %s
-
-// Test that we diagnose when the application hasn't initialized LLVM targets
-// supporting the MS-style inline asm parser.
-
-void Break() {
-  __asm { int 3 }
-}
-// CHECK: error: MS-style inline assembly is not available

Added: cfe/trunk/test/Tooling/ms-asm-no-target.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Tooling/ms-asm-no-target.cpp?rev=194249&view=auto
==============================================================================
--- cfe/trunk/test/Tooling/ms-asm-no-target.cpp (added)
+++ cfe/trunk/test/Tooling/ms-asm-no-target.cpp Fri Nov  8 00:50:48 2013
@@ -0,0 +1,13 @@
+// RUN: not clang-check "%s" -- -fasm-blocks -target x86_64-apple-darwin10 2>&1 | FileCheck -check-prefix=CHECK-X86 %s
+// RUN: not clang-check "%s" -- -fasm-blocks -target powerpc-apple-darwin10 2>&1 | FileCheck -check-prefix=CHECK-PPC %s
+
+// Test that we diagnose instead of crashing when the application hasn't
+// initialized LLVM targets supporting the MS-style inline asm parser.
+// Also test that the ordinary error is emitted on unsupported architectures.
+
+void Break() {
+  __asm { int 3 }
+}
+
+// CHECK-X86: error: MS-style inline assembly is not available
+// CHECK-PPC: error: Unsupported architecture 'powerpc' for MS-style inline assembly





More information about the cfe-commits mailing list