[clang] [llvm] [RISCV][MC] Implement MC for Base P extension (PR #123271)
via llvm-commits
llvm-commits at lists.llvm.org
Thu Feb 13 19:23:51 PST 2025
realqhc wrote:
> Can we break this up into smaller patches? The P spec is rather large. Reviewing 1000 lines of RISCVInstrInfoP.td in a single PR is very time consuming and exhausting as a reviewer.
>
> The changes to RISCVFeatures.td and RISCVInstrInfoZb.td and other things that go with that could be the first patch. That should be easy to review and get landed.
I've created a separate patch https://github.com/llvm/llvm-project/pull/127160 with the changes.
https://github.com/llvm/llvm-project/pull/123271
More information about the llvm-commits
mailing list