[llvm] r230885 - Revert r230655, "gold-plugin: "Upgrade" debug info and handle its warnings."

NAKAMURA Takumi geek4civic at gmail.com
Sat Feb 28 20:16:29 PST 2015


Author: chapuni
Date: Sat Feb 28 22:16:28 2015
New Revision: 230885

URL: http://llvm.org/viewvc/llvm-project?rev=230885&view=rev
Log:
Revert r230655, "gold-plugin: "Upgrade" debug info and handle its warnings."

It emits *millions of warnings* during selfhosting LTO build, to choke the buildbot with gigbytes of log.

Removed:
    llvm/trunk/test/tools/gold/Inputs/drop-debug.bc
    llvm/trunk/test/tools/gold/drop-debug.ll
Modified:
    llvm/trunk/tools/gold/gold-plugin.cpp

Removed: llvm/trunk/test/tools/gold/Inputs/drop-debug.bc
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/tools/gold/Inputs/drop-debug.bc?rev=230884&view=auto
==============================================================================
Binary files llvm/trunk/test/tools/gold/Inputs/drop-debug.bc (original) and llvm/trunk/test/tools/gold/Inputs/drop-debug.bc (removed) differ

Removed: llvm/trunk/test/tools/gold/drop-debug.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/tools/gold/drop-debug.ll?rev=230884&view=auto
==============================================================================
--- llvm/trunk/test/tools/gold/drop-debug.ll (original)
+++ llvm/trunk/test/tools/gold/drop-debug.ll (removed)
@@ -1,8 +0,0 @@
-; RUN: %gold -plugin %llvmshlibdir/LLVMgold.so \
-; RUN:    --plugin-opt=emit-llvm -shared %p/Inputs/drop-debug.bc \
-; RUN:    -o t2.bc 2>&1 | FileCheck %s
-
-; drop-debug.bc was created from "void f(void) {}" with clang 3.5 and
-; -gline-tables-only, so it contains old debug info.
-
-; CHECK: warning: LLVM gold plugin: ignoring debug info with an invalid version (1) in {{.*}}/Inputs/drop-debug.bc

Modified: llvm/trunk/tools/gold/gold-plugin.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/gold/gold-plugin.cpp?rev=230885&r1=230884&r2=230885&view=diff
==============================================================================
--- llvm/trunk/tools/gold/gold-plugin.cpp (original)
+++ llvm/trunk/tools/gold/gold-plugin.cpp Sat Feb 28 22:16:28 2015
@@ -20,7 +20,6 @@
 #include "llvm/Bitcode/ReaderWriter.h"
 #include "llvm/CodeGen/Analysis.h"
 #include "llvm/CodeGen/CommandFlags.h"
-#include "llvm/IR/AutoUpgrade.h"
 #include "llvm/IR/Constants.h"
 #include "llvm/IR/DiagnosticInfo.h"
 #include "llvm/IR/DiagnosticPrinter.h"
@@ -274,11 +273,11 @@ static bool shouldSkip(uint32_t Symflags
 }
 
 static void diagnosticHandler(const DiagnosticInfo &DI, void *Context) {
-  if (const auto *BDI = dyn_cast<BitcodeDiagnosticInfo>(&DI)) {
-    std::error_code EC = BDI->getError();
-    if (EC == BitcodeError::InvalidBitcodeSignature)
-      return;
-  }
+  assert(DI.getSeverity() == DS_Error && "Only expecting errors");
+  const auto &BDI = cast<BitcodeDiagnosticInfo>(DI);
+  std::error_code EC = BDI.getError();
+  if (EC == BitcodeError::InvalidBitcodeSignature)
+    return;
 
   std::string ErrStorage;
   {
@@ -286,21 +285,8 @@ static void diagnosticHandler(const Diag
     DiagnosticPrinterRawOStream DP(OS);
     DI.print(DP);
   }
-  ld_plugin_level Level;
-  switch (DI.getSeverity()) {
-  case DS_Error:
-    message(LDPL_FATAL, "LLVM gold plugin has failed to create LTO module: %s",
-            ErrStorage.c_str());
-    llvm_unreachable("Fatal doesn't return.");
-  case DS_Warning:
-    Level = LDPL_WARNING;
-    break;
-  case DS_Remark:
-  case DS_Note:
-    Level = LDPL_INFO;
-    break;
-  }
-  message(Level, "LLVM gold plugin: %s",  ErrStorage.c_str());
+  message(LDPL_FATAL, "LLVM gold plugin has failed to create LTO module: %s",
+          ErrStorage.c_str());
 }
 
 /// Called by gold to see whether this file is one that our plugin can handle.
@@ -575,7 +561,7 @@ static void freeSymName(ld_plugin_symbol
 
 static std::unique_ptr<Module>
 getModuleForFile(LLVMContext &Context, claimed_file &F,
-                 ld_plugin_input_file &Info, raw_fd_ostream *ApiFile,
+                 off_t Filesize, raw_fd_ostream *ApiFile,
                  StringSet<> &Internalize, StringSet<> &Maybe) {
 
   if (get_symbols(F.handle, F.syms.size(), &F.syms[0]) != LDPS_OK)
@@ -585,8 +571,7 @@ getModuleForFile(LLVMContext &Context, c
   if (get_view(F.handle, &View) != LDPS_OK)
     message(LDPL_FATAL, "Failed to get a view of file");
 
-  MemoryBufferRef BufferRef(StringRef((const char *)View, Info.filesize),
-                            Info.name);
+  MemoryBufferRef BufferRef(StringRef((const char *)View, Filesize), "");
   ErrorOr<std::unique_ptr<object::IRObjectFile>> ObjOrErr =
       object::IRObjectFile::create(BufferRef, Context);
 
@@ -598,8 +583,6 @@ getModuleForFile(LLVMContext &Context, c
 
   Module &M = Obj.getModule();
 
-  UpgradeDebugInfo(M);
-
   SmallPtrSet<GlobalValue *, 8> Used;
   collectUsedGlobalVariables(M, Used, /*CompilerUsed*/ false);
 
@@ -809,8 +792,6 @@ static ld_plugin_status allSymbolsReadHo
     return LDPS_OK;
 
   LLVMContext Context;
-  Context.setDiagnosticHandler(diagnosticHandler);
-
   std::unique_ptr<Module> Combined(new Module("ld-temp.o", Context));
   Linker L(Combined.get());
 
@@ -823,7 +804,8 @@ static ld_plugin_status allSymbolsReadHo
     if (get_input_file(F.handle, &File) != LDPS_OK)
       message(LDPL_FATAL, "Failed to get file information");
     std::unique_ptr<Module> M =
-        getModuleForFile(Context, F, File, ApiFile, Internalize, Maybe);
+        getModuleForFile(Context, F, File.filesize, ApiFile,
+                         Internalize, Maybe);
     if (!options::triple.empty())
       M->setTargetTriple(options::triple.c_str());
     else if (M->getTargetTriple().empty()) {





More information about the llvm-commits mailing list