[llvm] [RISCV] Move RISCVInsertVSETVLI to after phi elimination (PR #91440)

Philip Reames via llvm-commits llvm-commits at lists.llvm.org
Wed May 15 13:40:53 PDT 2024


preames wrote:

> I see this for both vmv.s.x and vmv.v.x. It looks like maybe we broke the don't care on tail policy somehow?

I think I see what happened here.  InitUndef is replacing undef/noreg/implicit_def operands with well defined sources, and we're no longer considering them to have a undef passthrough.  

The downstream effect of this is magnified by another pass we have downstream, so not your problem.  But I think this does explain some of the diffs I called out in the other review.  Can we plausible move InitUndef to just after the new placement here?

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


More information about the llvm-commits mailing list