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

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 3 10:20:19 PDT 2017


RKSimon added inline comments.


================
Comment at: lib/Target/X86/AsmParser/X86AsmParser.cpp:571
         } else {
-          State = IES_ERROR;
+          ErrMsg = "Invalid operand of multiplucation";
+          return true;
----------------
(typo) ErrMsg = "Invalid operand of multiplication";


================
Comment at: lib/Target/X86/AsmParser/X86AsmParser.cpp:1488
+                                    AsmToken::TokenKind::Star))
+          return Error(IdentLoc, ErrMsg);
       }
----------------
clang-format


================
Comment at: lib/Target/X86/AsmParser/X86AsmParser.cpp:1514
+                                      AsmToken::TokenKind::Star))
+            return Error(Loc, ErrMsg);
           End = consumeToken();
----------------
clang-format


https://reviews.llvm.org/D35621





More information about the llvm-commits mailing list