[llvm] [RISCV][VLOPT] Allow users that are passthrus if tail elements aren't demanded (PR #124066)

Philip Reames via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 23 09:16:54 PST 2025


https://github.com/preames requested changes to this pull request.

Hold on, this is now an unbounded recursion.  This can cause both stack explosion, and more likely, significant compile time regressions.  

I think you can achieve the same thing by adding a transform which uses the result of checkUsers to clear a pass thru to NoRegister.  (That is, convert an instruction from TA/TU to tail undefined.)  When we visit that instruction, it would propagate up.  

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


More information about the llvm-commits mailing list