[PATCH] D22585: [MC][X86] Fix Intel Operand assembly parsing for .set ids
Reid Kleckner via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 26 16:32:59 PDT 2016
rnk added inline comments.
================
Comment at: lib/Target/X86/AsmParser/X86AsmParser.cpp:1439
@@ +1438,3 @@
+ if (SM.getSym()) {
+ Error(Start, "cannot use more than one symbol in memory operand");
+ return nullptr;
----------------
Can you add a few negative tests exercising these errors? Look at how intel-syntax-ambiguous.s tests erroneous input.
https://reviews.llvm.org/D22585
More information about the llvm-commits
mailing list