[PATCH] D23303: LiveIntervalAnalysis: fix a crash in repairOldRegInRange

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 5 16:20:00 PST 2016


MatzeB added a comment.

Reviewing this because of http://llvm.org/PR31204.

I think this should be:

  if (LII != LR.end() && LII->start < endIdx)
    lastUseIdx = LII->end;
  else if (LII == LR.begin()) {
    // We may not have a liverange at all if this is a subregister untouched
    // between \p Begin and \p End.
  } else
    --LII;

Maybe you can test/update this before it gets pulled into 3.9.x?


Repository:
  rL LLVM

https://reviews.llvm.org/D23303





More information about the llvm-commits mailing list