[cfe-dev] Patch for IdentifierResolver

Argiris Kirtzidis akyrtzi at gmail.com
Wed Apr 9 23:51:59 PDT 2008


Attached new patch with the suggestions.

Chris Lattner wrote:
> What about "std::list<IdDeclInfo[512]>" (using a struct to hold the 
> array?)?  This saves an extra allocation of the list node, which is 
> not a big perf/memory win, but is tidier because you don't have to 
> delete[] the nodes.

This is tidier indeed, but won't there be coping of the struct when it 
enters the container, thus making it a bit inefficient ?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: idresolver2.patch
Type: text/x-diff
Size: 17591 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20080409/f506b96a/attachment.patch>


More information about the cfe-dev mailing list