Displaying 5 results from an estimated 5 matches for "r163107".
2012 Sep 04
0
[LLVMdev] Assert in LiveInterval update
On Sep 2, 2012, at 11:52 PM, Lang Hames <lhames at gmail.com> wrote:
> Hi Sergei,
>
> I just fixed the broken test case for PR13719 with r163107, but from the debugging output you've posted it suspect it won't fix your test case.
>
> Your analysis looks good - findLastUseBefore(..) doesn't appear to be handling physregs. I'm surprised that isn't causing more failures. I'll see if I can find a failing case in t...
2012 Sep 03
2
[LLVMdev] Assert in LiveInterval update
Hi Sergei,
I just fixed the broken test case for PR13719 with r163107, but from the
debugging output you've posted it suspect it won't fix your test case.
Your analysis looks good - findLastUseBefore(..) doesn't appear to be
handling physregs. I'm surprised that isn't causing more failures. I'll see
if I can find a failing case in the LLVM te...
2012 Sep 10
3
[LLVMdev] Assert in LiveInterval update
...subtle bugs.
Cheers,
Lang.
On Wed, Sep 5, 2012 at 4:07 AM, Jakob Stoklund Olesen <stoklund at 2pi.dk>wrote:
>
> On Sep 2, 2012, at 11:52 PM, Lang Hames <lhames at gmail.com> wrote:
>
> > Hi Sergei,
> >
> > I just fixed the broken test case for PR13719 with r163107, but from the
> debugging output you've posted it suspect it won't fix your test case.
> >
> > Your analysis looks good - findLastUseBefore(..) doesn't appear to be
> handling physregs. I'm surprised that isn't causing more failures. I'll see
> if I can...
2012 Aug 31
0
[LLVMdev] Assert in LiveInterval update
Lang,
I think I am getting closer to understanding this. The findLastUseBefore()
should probably look something like this:
// Return the last use of reg between NewIdx and OldIdx.
SlotIndex findLastUseBefore(unsigned Reg, SlotIndex OldIdx) {
SlotIndex LastUse = NewIdx;
if (TargetRegisterInfo::isPhysicalRegister(Reg)) {
for (MCRegUnitRootIterator Roots(Reg,
2012 Aug 31
2
[LLVMdev] Assert in LiveInterval update
Hi Lang,
Just one more quick question. in LiveIntervalAnalysis.cpp In
SlotIndex findLastUseBefore(unsigned Reg, SlotIndex OldIdx)
Did you really mean to use
for (MachineRegisterInfo::use_nodbg_iterator
UI = MRI.use_nodbg_begin(Reg),
UE = MRI.use_nodbg_end();
UI != UE; UI.skipInstruction()) {}
Aren't we currently dealing with units,