MaskRay wrote:

The fix is in llvm/lib/MC, but the test was added as 
clang/test/CodeGen/inline-asm-pending-errors.c with inappropriate comment.

MC behavior should be tested at the llvm layer — llvm/test/MC, or an llc 
-filetype=obj test if we need the inline-asm path to reach layout after a parse 
error. Clang tests that exercise the integrated assembler put coverage in the 
wrong place and are harder to maintain.

Already fixed.

https://github.com/llvm/llvm-project/pull/225769
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to