diff options
| author | James Molloy <james.molloy@arm.com> | 2014-07-25 10:19:47 +0000 |
|---|---|---|
| committer | James Molloy <james.molloy@arm.com> | 2014-07-25 10:19:47 +0000 |
| commit | 8a157bf8f2117371bd9317b3eaf38956afe88781 (patch) | |
| tree | e64eb91f015eb05d34e3cf28c512e2b76d4da0c2 /clang/test/Sema/inline-asm-validate.c | |
| parent | b4553da4814e0f0fe8335eda3347c8613f2ece3b (diff) | |
| download | bcm5719-llvm-8a157bf8f2117371bd9317b3eaf38956afe88781.tar.gz bcm5719-llvm-8a157bf8f2117371bd9317b3eaf38956afe88781.zip | |
Revert part of r206963
Specifically the part where we removed a warning to be compatible with GCC, which has been widely regarded as a bad idea.
I'm not quite happy with how obtuse this warning is, especially in the fairly common case of a 32-bit integer literal, so I've got another patch awaiting review that adds a fixit to reduce confusion.
llvm-svn: 213935
Diffstat (limited to 'clang/test/Sema/inline-asm-validate.c')
| -rw-r--r-- | clang/test/Sema/inline-asm-validate.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/clang/test/Sema/inline-asm-validate.c b/clang/test/Sema/inline-asm-validate.c index c32dedb65e8..6fa760c8092 100644 --- a/clang/test/Sema/inline-asm-validate.c +++ b/clang/test/Sema/inline-asm-validate.c @@ -1,9 +1,8 @@ // RUN: %clang_cc1 -triple arm64-apple-macosx10.8.0 -fsyntax-only -verify %s -// expected-no-diagnostics unsigned t, r, *p; int foo (void) { - __asm__ __volatile__( "stxr %w[_t], %[_r], [%[_p]]" : [_t] "=&r" (t) : [_p] "p" (p), [_r] "r" (r) : "memory"); + __asm__ __volatile__( "stxr %w[_t], %[_r], [%[_p]]" : [_t] "=&r" (t) : [_p] "p" (p), [_r] "r" (r) : "memory"); // expected-warning{{value size does not match register size specified by the constraint and modifier}} return 1; } |
