[PATCH] Allocate MCSymbol Name intrusively only when required

Pete Cooper peter_cooper at apple.com
Tue Jun 9 10:36:07 PDT 2015


Updated to NameEntryTy and *&.


> On Jun 8, 2015, at 2:41 PM, Pete Cooper <peter_cooper at apple.com> wrote:
> 
> 
>> On Jun 8, 2015, at 2:39 PM, David Blaikie <dblaikie at gmail.com <mailto:dblaikie at gmail.com>> wrote:
>> 
>> (though perhaps these names have precedence from the types you modeled them off, I'm not sure)
> Not particularly.  NameEntryTy is a good name.
> 
> I like all the feedback.  Will update the patch accordingly.
> 
> Thanks,
> Pete
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150609/48d22106/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mcsymbol-name.diff
Type: application/octet-stream
Size: 6143 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150609/48d22106/attachment.obj>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150609/48d22106/attachment-0001.html>


More information about the llvm-commits mailing list