[llvm] [RISCV] Merge Xqci Decoder Tables (PR #128140)

Philip Reames via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 21 07:35:05 PST 2025


preames wrote:

> > @lenary do you want to land this first and I'll rebase #128102
> 
> No. please land yours first. I intend to rebase over the Xqcilia patch and work out if we're going to have encoding difficulties when trying to fit instructions into minor gaps in the encoding space.

If I'm reading this right, that would be a non-conforming extension.  The conflict wouldn't be with other qc extensions, but with the standard ones right?  If so, having a single qc table should still be fine, you just need to check it before the standard table.  Though, having the qc table split into conforming and non-conforming also doesn't seem unreasonable, which is what I think you meant.  So either option works.  :)

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


More information about the llvm-commits mailing list