[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