[PATCH] D26043: Replace GAlloc with a template function.

Rafael Ávila de Espíndola via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 28 07:49:13 PDT 2016


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

LGTM with nit



================
Comment at: ELF/Memory.cpp:23
+
+void elf::freeArena() {
+  for (SpecificAllocBase *Alloc : SpecificAllocBase::Instances)
----------------
Do we still need to free this at a specific point in time? If not just letting the destructors take care of it should simplify the code. 


https://reviews.llvm.org/D26043





More information about the llvm-commits mailing list