https://github.com/topperc commented:
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. 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
