[PATCH] D82916: LIS: fix handleMove to properly extend main range

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 7 09:49:09 PDT 2020


rampitec added a comment.

In D82916#2136518 <https://reviews.llvm.org/D82916#2136518>, @vpykhtin wrote:

> Given than the condition here is rare, I'm ok with the patch, reworking undef handling in scheduler is a massive work.


Right. Scheduler deliberately drops undef flags before the scheduling. To recreate them in the middle of the scheduling would be against the concept. I assume there is nothing after the scheduler which needs them. Moreover it looks like complete recompute of an LI in the middle of the scheduling is expensive and recompute of just some will simply result in an incosistent LIS.


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

https://reviews.llvm.org/D82916





More information about the llvm-commits mailing list