[llvm] [RISC-V][MC] Validate instructions prior to emission in AsmParser (PR #171739)

Alexander Richardson via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 10 21:07:32 PST 2025


arichardson wrote:

> IMO this should be behind a developer-oriented flag, like `-verify-machineinstrs` is for llc today.
> 
> The logic looks right, and yes sharing between targets would be good but can come later.

Thanks, that's a good suggestion. How about only doing it for assertions builds or EXPENSIVE_CHECKS? I haven't measured the performance impact here but since it's only for assembly inputs it shouldn't be on the critical path for most projects.

https://github.com/llvm/llvm-project/pull/171739


More information about the llvm-commits mailing list