[llvm-commits] [llvm] r150552 - in /llvm/trunk: include/llvm/CodeGen/LiveIntervalAnalysis.h lib/CodeGen/LiveIntervalAnalysis.cpp lib/CodeGen/MachineScheduler.cpp

Lang Hames lhames at gmail.com
Wed Feb 15 14:50:41 PST 2012


Right you are. Fixed in r150627.

Cheers,
Lang.

On Wed, Feb 15, 2012 at 9:33 AM, Jakob Stoklund Olesen <stoklund at 2pi.dk>wrote:

>
> On Feb 14, 2012, at 5:23 PM, Lang Hames <lhames at gmail.com> wrote:
>
> >   SlotIndex origIdx = indexes_->getInstructionIndex(mi);
>
> > +  MachineBasicBlock* mbb = mi->getParent();
> > +
> > +  assert(getMBBFromIndex(origIdx) == mbb &&
> > +         "Cannot handle moves across basic block boundaries.");
>
> That assert probably doesn't do what you want. From SlotIndexes.h:
>
>    MachineBasicBlock* getMBBFromIndex(SlotIndex index) const {
>      if (MachineInstr *MI = getInstructionFromIndex(index))
>        return MI->getParent();
>
> You could use getMBBRange() instead.
>
> /jakob
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120215/3a604935/attachment.html>


More information about the llvm-commits mailing list