LucasChollet wrote:

> Not related to this but does the GPR code work with GPR pairs used f64 with 
> Zdinx?

No yet -> #206323

> Need test with larger LMUL arguments.

Good catch, the current implementation doesn't work when LMUL > 1. I think I 
want to use the same approach as for GPRPair, so splitting V registers into 
there underlying physical regs for isUsed/IsArgumentRegister analysis.

Using this approach means that we would emit a `ClearVReg` for each physical V 
reg. I imagine that the pseudo expansion pass could be smart about them and 
merge some of these instructions by leveraging LMUL (as suggested 
[here](https://github.com/llvm/llvm-project/pull/206206#discussion_r3484752653)),
 but would that be required for landing this patch? Seems to me like an 
optimization that could be added later.

https://github.com/llvm/llvm-project/pull/206206
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to