<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Oct 28, 2016 at 10:58 AM, Mehdi AMINI <span dir="ltr"><<a href="mailto:mehdi.amini@apple.com" target="_blank">mehdi.amini@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">mehdi_amini added inline comments.<br>
<br>
<br>
================<br>
<span class="">Comment at: ELF/Memory.h:31<br>
+extern llvm::BumpPtrAllocator BAlloc;<br>
+extern llvm::StringSaver Saver;<br>
+<br>
----------------<br>
</span><span class="">ruiu wrote:<br>
> mehdi_amini wrote:<br>
> > Global mutable state never seems like a good idea.<br>
> I disagree. It depends on what you are doing and what your program is.<br>
</span>I believe that in an ecosystem where we want to be able to evolve and innovate, like the LLVM ecosystem, this prevents code reuse, modularity, and future innovation. This is just unacceptable technical debt.<br>
Again we had this debate about lld, but this is just another opportunity to express another strong disagreement with what's going on in lld.<br></blockquote><div><br></div><div>I appreciate you review my patch, but your comment raises a vague concern which doesn't have a justification. We do care about the LLVM ecosystem, modularity and future innovation (I believe you would find LLD is *very* hackable if you read the source code), and based on our experience and knowledge with this codebase, we agreed that an arena allocator-ish memory management will fit best.</div></div></div></div>