[PATCH] IR: Add specialized debug info metadata nodes

Duncan P. N. Exon Smith dexonsmith at apple.com
Wed Feb 4 15:46:44 PST 2015


> On 2015-Feb-04, at 15:37, Frédéric Riss <friss at apple.com> wrote:
> 
>>> 
>>> - The word 'context' is overloaded: `MDNode::getContext()` already
>>>   exists, and returns an `LLVMContext&`; `DIDescriptor` uses 'context'
>>>   to mean "the node that this one is defined inside".  I chose the
>>>   word 'parent' instead of 'context' here.  Is this word okay?  If
>>>   not, what about 'scope'?  This will be reflected in the assembly
>>>   changes to come (I'd like the C++ names to match the assembly names,
>>>   although technically it's not necessary).
>>> 
>>> I'd /probably/ go with scope (we already have scope in the MDLocations, so that seems consistent), but fairly on-the-fence.
>>> 
>> 
>> Weirdly, I didn't even notice that :).  In that case I like 'scope'
>> better too.  I'll update to that before commit.
> 
> Seems most natural. Can the futur getScope() return something that doesn’t derive from MDScope?

I don't think so.



More information about the llvm-commits mailing list