<p dir="ltr">Lgtm, but can you add a test?</p>
<div class="gmail_quote">On Apr 19, 2016 12:10 PM, "Teresa Johnson" <<a href="mailto:tejohnson@google.com">tejohnson@google.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">tejohnson created this revision.<br>
tejohnson added reviewers: rafael, joker-eph.<br>
tejohnson added a subscriber: llvm-commits.<br>
<br>
Applies Mehdi's optimization (r263086) to disable value names other than<br>
for GlobalValues to LTO/ThinLTO performed via the gold-plugin, in the<br>
same manner as it is applied in libLTO.<br>
<br>
<a href="http://reviews.llvm.org/D19269" rel="noreferrer" target="_blank">http://reviews.llvm.org/D19269</a><br>
<br>
Files:<br>
  tools/gold/gold-plugin.cpp<br>
<br>
Index: tools/gold/gold-plugin.cpp<br>
===================================================================<br>
--- tools/gold/gold-plugin.cpp<br>
+++ tools/gold/gold-plugin.cpp<br>
@@ -167,8 +167,10 @@<br>
   static unsigned Parallelism = 0;<br>
 #ifdef NDEBUG<br>
   static bool DisableVerify = true;<br>
+  static bool DiscardValueNames = true;<br>
 #else<br>
   static bool DisableVerify = false;<br>
+  static bool DiscardValueNames = false;<br>
 #endif<br>
   static std::string obj_path;<br>
   static std::string extra_library_path;<br>
@@ -225,6 +227,8 @@<br>
         message(LDPL_FATAL, "Invalid parallelism level: %s", opt_ + 5);<br>
     } else if (opt == "disable-verify") {<br>
       DisableVerify = true;<br>
+    } else if (opt == "discard-value-names") {<br>
+      DiscardValueNames = true;<br>
     } else {<br>
       // Save this option to pass to the code generator.<br>
       // ParseCommandLineOptions() expects argv[0] to be program name. Lazily<br>
@@ -1114,6 +1118,7 @@<br>
                                raw_fd_ostream *OS, unsigned TaskID) {<br>
   // Need to use a separate context for each task<br>
   LLVMContext Context;<br>
+  Context.setDiscardValueNames(options::DiscardValueNames);<br>
   Context.enableDebugTypeODRUniquing(); // Merge debug info types.<br>
   Context.setDiagnosticHandler(diagnosticHandlerForContext, nullptr, true);<br>
<br>
@@ -1236,6 +1241,7 @@<br>
   }<br>
<br>
   LLVMContext Context;<br>
+  Context.setDiscardValueNames(options::DiscardValueNames);<br>
   Context.enableDebugTypeODRUniquing(); // Merge debug info types.<br>
   Context.setDiagnosticHandler(diagnosticHandlerForContext, nullptr, true);<br>
<br>
<br>
<br>
</blockquote></div>