[PATCH] D19660: Use a single context for lto

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 28 10:50:40 PDT 2016


ruiu accepted this revision.
ruiu added a comment.

LGTM with this change.


================
Comment at: ELF/Driver.cpp:265-269
@@ -264,1 +264,7 @@
 
+  // This is a flag to discard all but GlobalValue names.
+  // We want to enable it by default because it saves memory.
+  // Disable it only when a developer option (-save-temps) is given.
+  Context.setDiscardValueNames(!Config->SaveTemps);
+  Context.enableDebugTypeODRUniquing();
+
----------------
I'd move this to initLLVM and swap the order of initLLVM and readConfigs.


http://reviews.llvm.org/D19660





More information about the llvm-commits mailing list