craig.topper added a comment.

In D138807#3962417 <https://reviews.llvm.org/D138807#3962417>, @craig.topper 
wrote:

> Anything that uses OP-P needs to have a DecoderNamespace assigned and 
> RISCVDisassembler::getInstruction will need to lookup that table when V/Zve 
> is enabled. Otherwise we will have a conflict if the P extension is ever 
> commited to LLVM.

I think we can ignore the OP-P conflict until P is ratified and added to LLVM.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D138807/new/

https://reviews.llvm.org/D138807

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to