[llvm-commits] [llvm] r139581 - in /llvm/trunk/lib/CodeGen: LiveRangeCalc.cpp LiveRangeCalc.h

Jakob Stoklund Olesen stoklund at 2pi.dk
Tue Sep 13 09:06:17 PDT 2011


On Sep 13, 2011, at 8:58 AM, Eli Friedman wrote:

> On Tue, Sep 13, 2011 at 8:51 AM, Jakob Stoklund Olesen <stoklund at 2pi.dk> wrote:
>> 
>> On Sep 12, 2011, at 8:58 PM, NAKAMURA Takumi wrote:
>>> -      LiveOut[MBB] = LiveOutPair(VNI, 0);
>>> +      LiveOut[MBB] = LiveOutPair(VNI, (MachineDomTreeNode *)0);
>> 
>> Thanks, Takumi.
>> 
>> Do you know if NULL would work too?
> 
> NULL doesn't help... it's always #defined to 0.  (That isn't precisely
> correct, but close enough.)

Wonderful, thanks.

In the past, I've had to resort to inline functions to work around this problem:

static inline LiveOutPair MakeLOP(VNInfo *a, MachineDomTreeNode *b) {
  return LiveOutPair(a, b);
}

In this case, it only happens twice, and one is in a header, so I'll just leave it as is.

/jakob




More information about the llvm-commits mailing list