Correctly handle -pass-remarks in the gold plugin.
llvm-svn: 231132
This commit is contained in:
parent
af0f45f1c9
commit
84483d247f
|
@ -0,0 +1,17 @@
|
||||||
|
; RUN: llvm-as %s -o %t.o
|
||||||
|
|
||||||
|
; RUN: %gold -plugin %llvmshlibdir/LLVMgold.so \
|
||||||
|
; RUN: -plugin-opt=-pass-remarks=inline %t.o -o %t2.o 2>&1 | FileCheck %s
|
||||||
|
|
||||||
|
; CHECK: f inlined into _start
|
||||||
|
target datalayout = "e-m:e-i64:64-f80:128-n8:16:32:64-S128"
|
||||||
|
target triple = "x86_64-unknown-linux-gnu"
|
||||||
|
|
||||||
|
define i32 @f() {
|
||||||
|
ret i32 0
|
||||||
|
}
|
||||||
|
|
||||||
|
define i32 @_start() {
|
||||||
|
%call = call i32 @f()
|
||||||
|
ret i32 %call
|
||||||
|
}
|
|
@ -296,12 +296,9 @@ static void diagnosticHandler(const DiagnosticInfo &DI, void *Context) {
|
||||||
Level = LDPL_WARNING;
|
Level = LDPL_WARNING;
|
||||||
break;
|
break;
|
||||||
case DS_Note:
|
case DS_Note:
|
||||||
|
case DS_Remark:
|
||||||
Level = LDPL_INFO;
|
Level = LDPL_INFO;
|
||||||
break;
|
break;
|
||||||
// FIXME: Just ignore remarks for now. They are always passed by
|
|
||||||
// if there is a custom diagnostic handler, so we get flooded.
|
|
||||||
case DS_Remark:
|
|
||||||
return;
|
|
||||||
}
|
}
|
||||||
message(Level, "LLVM gold plugin: %s", ErrStorage.c_str());
|
message(Level, "LLVM gold plugin: %s", ErrStorage.c_str());
|
||||||
}
|
}
|
||||||
|
@ -812,7 +809,7 @@ static ld_plugin_status allSymbolsReadHook(raw_fd_ostream *ApiFile) {
|
||||||
return LDPS_OK;
|
return LDPS_OK;
|
||||||
|
|
||||||
LLVMContext Context;
|
LLVMContext Context;
|
||||||
Context.setDiagnosticHandler(diagnosticHandler);
|
Context.setDiagnosticHandler(diagnosticHandler, nullptr, false);
|
||||||
|
|
||||||
std::unique_ptr<Module> Combined(new Module("ld-temp.o", Context));
|
std::unique_ptr<Module> Combined(new Module("ld-temp.o", Context));
|
||||||
Linker L(Combined.get());
|
Linker L(Combined.get());
|
||||||
|
|
Loading…
Reference in New Issue