================ @@ -11,7 +11,7 @@ ! V9: unknown membar tag membar #BadTag - ! V8: instruction requires a CPU feature not currently enabled + ! V8: unexpected token ! V9: invalid membar mask number membar -127 ---------------- s-barannikov wrote:
There are many places where ParseStatus is not handled correctly. In this case, it is ``` if (!parseOperand(Operands, Name).isSuccess()) { SMLoc Loc = getLexer().getLoc(); return Error(Loc, "unexpected token"); } ``` It should propagate the error returned by parseOperand instead of issuing another one. Something like: ``` ParseStatus R = parseOperand(Operands, Name).isSuccess(); if (R.isFailure()) return true; if (R.isNoMatch()) return TokError( "unexpected token"); // success, proceed do the next operand ``` https://github.com/llvm/llvm-project/pull/96021 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits