[PATCH] D35621: X86 Asm can't work properly with symbolic Scale

Andrew V. Tischenko via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 15 05:12:22 PDT 2017


avt77 updated this revision to Diff 111146.
avt77 added a comment.
Herald added a subscriber: eraman.

The Clang changes added.


https://reviews.llvm.org/D35621

Files:
  test/Sema/ms-inline-asm.c


Index: test/Sema/ms-inline-asm.c
===================================================================
--- test/Sema/ms-inline-asm.c
+++ test/Sema/ms-inline-asm.c
@@ -80,12 +80,12 @@
 } A;
 
 void t3() {
-  __asm { mov eax, [eax] UndeclaredId } // expected-error {{unknown token in expression}} expected-error {{use of undeclared label 'UndeclaredId'}}
+  __asm { mov eax, [eax] UndeclaredId } // expected-error {{invalid address operation}} expected-error {{use of undeclared label 'UndeclaredId'}}
 
   // FIXME: Only emit one diagnostic here.
   // expected-error at +3 {{use of undeclared label 'A'}}
   // expected-error at +2 {{unexpected type name 'A': expected expression}}
-  // expected-error at +1 {{unknown token in expression}}
+  // expected-error at +1 {{invalid address operation}}
   __asm { mov eax, [eax] A }
 }
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D35621.111146.patch
Type: text/x-patch
Size: 819 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170815/7e14a07a/attachment.bin>


More information about the llvm-commits mailing list