clayborg accepted this revision.
clayborg added a comment.
This revision is now accepted and ready to land.

In http://reviews.llvm.org/D15437#308839, @clayborg wrote:

> Can we change DebugMacroEntry to contain a std::unique_ptr<DebugMacro> 
> instead of a shared pointer? This would save us one full pointer in each 
> DebugMacroEntry.
>
> That is the only change I would like to see.


Actually never mind. I believe the whole reason this shared pointer is here it 
to share the object between multiple other objects...


http://reviews.llvm.org/D15437



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to