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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits