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

Reply via email to