[llvm] [RISCV] Copy base instruction TSFlags in RVV pseudo TSFlags. NFC (PR #149785)

Luke Lau via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 22 00:35:33 PDT 2025


lukel97 wrote:

> > To do this I needed to merge Pseudo and RISCVVPseudo so the latter extends the former.
> 
> I support the merging for cleanliness, but I'm not sure what about this change made it necessary.

I may be missing something, but in order to set the `ElementsDependOn`/`DestEEW` fields on Pseudo and access `BaseInstr` from RISCVVPseudo at the same time they needed to be in the same hierarchy. Tablegen complained when I tried to set those fields from RISCVVPseudo otherwise.

I'm going to rework this PR to leave out the TSFlags bit and just have the hierarchy change for now anyway.

https://github.com/llvm/llvm-project/pull/149785


More information about the llvm-commits mailing list