[PATCH] D90591: [ARM] Introduce t2DoLoopStartTP

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 2 07:52:36 PST 2020


dmgreen added a comment.

> That makes sense 👍 Do you envisage us moving all of the TP validation to the MVEVPTOptimisationsPass and then only doing the actual codegen in ARMLowOverheadLoops?

I was thinking of putting some bits here, there is some stuff about terminators that produce values that I was hoping to do here too. Also something about ensuring lr is used as a predicate. We can't do everything at this early stage as we have to be very careful about being able to handle loops that end up being reverted. Some things are better to do pre-RA though, like the changes here, and let the backend pass do what it can from there like removing the unused arg.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D90591/new/

https://reviews.llvm.org/D90591



More information about the llvm-commits mailing list