[PATCH] D26042: Consolidate BumpPtrAllocators.

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 28 10:55:34 PDT 2016


ruiu added inline comments.


================
Comment at: ELF/Memory.h:31
+extern llvm::BumpPtrAllocator BAlloc;
+extern llvm::StringSaver Saver;
+
----------------
mehdi_amini wrote:
> Global mutable state never seems like a good idea.
I disagree. It depends on what you are doing and what your program is.


https://reviews.llvm.org/D26042





More information about the llvm-commits mailing list