[llvm] [RISCV][NFC] Move Zawrs implementation to RISCVInstrInfoZawrs.td (PR #76940)
Wang Pengcheng via llvm-commits
llvm-commits at lists.llvm.org
Thu Jan 4 08:44:03 PST 2024
wangpc-pp wrote:
> > LGTM. I think it's not necessarily the case that every extension should have it's own file
>
> I agree. Adding a file for every extension doesn't scale well if we keep adding extensions at the rate we are.
What about adding it to `RISCVInstrInfoZa.td`? We will have `Zabha` (https://github.com/riscv/riscv-zabha) and may have `Zalasr` (https://github.com/mehnadnerd/riscv-zalasr) later.
And what is the policy? Should we put all (or these simple ones) `Zi*` extensions into `RISCVInstrInfoZi.td` for example? We already have `RISCVInstrInfoZb.td` and `RISCVInstrInfoZc.td`.
https://github.com/llvm/llvm-project/pull/76940
More information about the llvm-commits
mailing list