<div dir="ltr">Also, I don't like the boilerplate code that saves the old diagnostic handler, sets the new one, and then restores the old one after the use of the Linker.<div><br></div><div>Maybe add some kind of a scoped handler for the users that just need to handle an error from a single api call?</div><div><br></div><div>E.g.:</div><div><br></div><div>class ScopedDiagnosticRedirector {</div><div>public:<br></div><div>  ScopedDiagnosticRedirector(llvm::LLVMContext &Context, llvm::raw_ostream &Stream)</div><div>  Â  : Context(Context), Stream(Stream), OldDiagHandler(Context.getDiagnosticHandler()),</div><div>  Â  Â  OldDiagContext(Context.getDiagnosticContext()) {</div><div>  Â  Context.setDiagnosticHandler(diagHandler, this);</div><div>  }</div><div>  ~ScopedDiagnosticRedirector() {</div><div>  Â  Context.setDiagnosticHandler(OldDiagHandler, OldDiagContext);</div><div>  }</div><div><br></div><div>private:</div><div><div>  llvm::LLVMContext &Context;</div><div>  llvm::raw_ostream &Stream;</div><div>  llvm::LLVMContext::DiagnosticHandlerTy OldDiagHandler;</div><div>  void *OldDiagContext;</div><div><br></div><div>  static void diagHandler(const llvm::DiagnosticInfo &DI, void *Ctx) {</div><div>  Â  auto Redirector = reinterpret_cast<ScopedDiagnosticRedirector*>(Ctx);<br></div><div>  Â  llvm::DiagnosticPrinterRawOStream Printer(Redirector->Stream);</div><div>  Â  DI.print(Printer);</div><div>  }</div></div><div>};<br></div><div><br></div><div>WDYT?</div><div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Oct 27, 2014 at 12:52 PM, Alexander Kornienko <span dir="ltr"><<a href="mailto:alexfh@google.com" target="_blank">alexfh@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">This change makes the users of the relevant interfaces add a ton of boilerplate code in order to get an error. Can we add an adapter to make the transition easier? E.g. a diagnostic handler method that would get a pointer to a raw_ostream as the context which could then be used with either errs() or raw_string_ostream?<div><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Oct 24, 2014 at 9:06 PM, Rafael Espindola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Author: rafael<br>
Date: Fri Oct 24 23:06:10 2014<br>
New Revision: 220608<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=220608&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=220608&view=rev</a><br>
Log:<br>
Update the error handling of lib/Linker.<br>
<br>
Instead of passing a std::string&, use the new diagnostic infrastructure.<br>
<br>
Modified:<br>
  Â  llvm/trunk/include/llvm/IR/DiagnosticInfo.h<br>
  Â  llvm/trunk/include/llvm/LTO/LTOCodeGenerator.h<br>
  Â  llvm/trunk/include/llvm/Linker/Linker.h<br>
  Â  llvm/trunk/lib/LTO/LTOCodeGenerator.cpp<br>
  Â  llvm/trunk/lib/Linker/LinkModules.cpp<br>
  Â  llvm/trunk/tools/bugpoint/BugDriver.cpp<br>
  Â  llvm/trunk/tools/bugpoint/Miscompilation.cpp<br>
  Â  llvm/trunk/tools/gold/gold-plugin.cpp<br>
  Â  llvm/trunk/tools/llvm-link/llvm-link.cpp<br>
  Â  llvm/trunk/tools/llvm-lto/llvm-lto.cpp<br>
  Â  llvm/trunk/tools/lto/lto.cpp<br>
  Â  llvm/trunk/unittests/Linker/LinkModulesTest.cpp<br>
<br>
Modified: llvm/trunk/include/llvm/IR/DiagnosticInfo.h<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/IR/DiagnosticInfo.h?rev=220608&r1=220607&r2=220608&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/IR/DiagnosticInfo.h?rev=220608&r1=220607&r2=220608&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/include/llvm/IR/DiagnosticInfo.h (original)<br>
+++ llvm/trunk/include/llvm/IR/DiagnosticInfo.h Fri Oct 24 23:06:10 2014<br>
@@ -47,6 +47,7 @@ enum DiagnosticSeverity {<br>
 enum DiagnosticKind {<br>
  Â DK_InlineAsm,<br>
  Â DK_StackSize,<br>
+  DK_Linker,<br>
  Â DK_DebugMetadataVersion,<br>
  Â DK_SampleProfile,<br>
  Â DK_OptimizationRemark,<br>
<br>
Modified: llvm/trunk/include/llvm/LTO/LTOCodeGenerator.h<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/LTO/LTOCodeGenerator.h?rev=220608&r1=220607&r2=220608&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/LTO/LTOCodeGenerator.h?rev=220608&r1=220607&r2=220608&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/include/llvm/LTO/LTOCodeGenerator.h (original)<br>
+++ llvm/trunk/include/llvm/LTO/LTOCodeGenerator.h Fri Oct 24 23:06:10 2014<br>
@@ -64,7 +64,7 @@ struct LTOCodeGenerator {<br>
  Â ~LTOCodeGenerator();<br>
<br>
  Â // Merge given module, return true on success.<br>
-  bool addModule(struct LTOModule*, std::string &errMsg);<br>
+  bool addModule(struct LTOModule *);<br>
<br>
  Â void setTargetOptions(TargetOptions options);<br>
  Â void setDebugInfo(lto_debug_model);<br>
<br>
Modified: llvm/trunk/include/llvm/Linker/Linker.h<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Linker/Linker.h?rev=220608&r1=220607&r2=220608&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Linker/Linker.h?rev=220608&r1=220607&r2=220608&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/include/llvm/Linker/Linker.h (original)<br>
+++ llvm/trunk/include/llvm/Linker/Linker.h Fri Oct 24 23:06:10 2014<br>
@@ -11,14 +11,10 @@<br>
 #define LLVM_LINKER_LINKER_H<br>
<br>
 #include "llvm/ADT/SmallPtrSet.h"<br>
-#include <string><br>
<br>
 namespace llvm {<br>
<br>
-class Comdat;<br>
-class GlobalValue;<br>
 class Module;<br>
-class StringRef;<br>
 class StructType;<br>
<br>
 /// This class provides the core functionality of linking in LLVM. It keeps a<br>
@@ -32,7 +28,7 @@ class Linker {<br>
  Â  Â  Â PreserveSource = 1 // Preserve the source module.<br>
  Â  Â };<br>
<br>
-  Â  Linker(Module *M, bool SuppressWarnings=false);<br>
+  Â  Linker(Module *M);<br>
  Â  Â ~Linker();<br>
<br>
  Â  Â Module *getModule() const { return Composite; }<br>
@@ -43,19 +39,16 @@ class Linker {<br>
  Â  Â /// If \p ErrorMsg is not null, information about any error is written<br>
  Â  Â /// to it.<br>
  Â  Â /// Returns true on error.<br>
-  Â  bool linkInModule(Module *Src, unsigned Mode, std::string *ErrorMsg);<br>
-  Â  bool linkInModule(Module *Src, std::string *ErrorMsg) {<br>
-  Â  Â  return linkInModule(Src, Linker::DestroySource, ErrorMsg);<br>
+  Â  bool linkInModule(Module *Src, unsigned Mode);<br>
+  Â  bool linkInModule(Module *Src) {<br>
+  Â  Â  return linkInModule(Src, Linker::DestroySource);<br>
  Â  Â }<br>
<br>
-  Â  static bool LinkModules(Module *Dest, Module *Src, unsigned Mode,<br>
-  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  std::string *ErrorMsg);<br>
+  Â  static bool LinkModules(Module *Dest, Module *Src, unsigned Mode);<br>
<br>
  Â private:<br>
  Â  Â Module *Composite;<br>
  Â  Â SmallPtrSet<StructType*, 32> IdentifiedStructTypes;<br>
-<br>
-  Â  bool SuppressWarnings;<br>
 };<br>
<br>
 } // End llvm namespace<br>
<br>
Modified: llvm/trunk/lib/LTO/LTOCodeGenerator.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/LTO/LTOCodeGenerator.cpp?rev=220608&r1=220607&r2=220608&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/LTO/LTOCodeGenerator.cpp?rev=220608&r1=220607&r2=220608&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/LTO/LTOCodeGenerator.cpp (original)<br>
+++ llvm/trunk/lib/LTO/LTOCodeGenerator.cpp Fri Oct 24 23:06:10 2014<br>
@@ -113,8 +113,8 @@ void LTOCodeGenerator::initializeLTOPass<br>
  Â initializeCFGSimplifyPassPass(R);<br>
 }<br>
<br>
-bool LTOCodeGenerator::addModule(LTOModule* mod, std::string& errMsg) {<br>
-  bool ret = IRLinker.linkInModule(&mod->getModule(), &errMsg);<br>
+bool LTOCodeGenerator::addModule(LTOModule *mod) {<br>
+  bool ret = IRLinker.linkInModule(&mod->getModule());<br>
<br>
  Â const std::vector<const char*> &undefs = mod->getAsmUndefinedRefs();<br>
  Â for (int i = 0, e = undefs.size(); i != e; ++i)<br>
<br>
Modified: llvm/trunk/lib/Linker/LinkModules.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Linker/LinkModules.cpp?rev=220608&r1=220607&r2=220608&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Linker/LinkModules.cpp?rev=220608&r1=220607&r2=220608&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/Linker/LinkModules.cpp (original)<br>
+++ llvm/trunk/lib/Linker/LinkModules.cpp Fri Oct 24 23:06:10 2014<br>
@@ -17,6 +17,9 @@<br>
 #include "llvm/ADT/SetVector.h"<br>
 #include "llvm/ADT/SmallString.h"<br>
 #include "llvm/IR/Constants.h"<br>
+#include "llvm/IR/DiagnosticInfo.h"<br>
+#include "llvm/IR/DiagnosticPrinter.h"<br>
+#include "llvm/IR/LLVMContext.h"<br>
 #include "llvm/IR/Module.h"<br>
 #include "llvm/IR/TypeFinder.h"<br>
 #include "llvm/Support/CommandLine.h"<br>
@@ -376,6 +379,20 @@ namespace {<br>
  Â  Â Value *materializeValueFor(Value *V) override;<br>
  Â };<br>
<br>
+  namespace {<br>
+  class LinkDiagnosticInfo : public DiagnosticInfo {<br>
+  Â  const Twine &Msg;<br>
+<br>
+  public:<br>
+  Â  LinkDiagnosticInfo(DiagnosticSeverity Severity, const Twine &Msg);<br>
+  Â  void print(DiagnosticPrinter &DP) const override;<br>
+  };<br>
+  LinkDiagnosticInfo::LinkDiagnosticInfo(DiagnosticSeverity Severity,<br>
+  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â const Twine &Msg)<br>
+  Â  Â  : DiagnosticInfo(DK_Linker, Severity), Msg(Msg) {}<br>
+  void LinkDiagnosticInfo::print(DiagnosticPrinter &DP) const { DP << Msg; }<br>
+  }<br>
+<br>
  Â /// ModuleLinker - This is an implementation class for the LinkModules<br>
  Â /// function, which is the entrypoint for this file.<br>
  Â class ModuleLinker {<br>
@@ -406,29 +423,27 @@ namespace {<br>
  Â  Â // Vector of functions to lazily link in.<br>
  Â  Â std::vector<Function*> LazilyLinkFunctions;<br>
<br>
-  Â  bool SuppressWarnings;<br>
-<br>
  Â public:<br>
-  Â  std::string ErrorMsg;<br>
-<br>
-  Â  ModuleLinker(Module *dstM, TypeSet &Set, Module *srcM, unsigned mode,<br>
-  Â  Â  Â  Â  Â  Â  Â  Â bool SuppressWarnings=false)<br>
+  Â  ModuleLinker(Module *dstM, TypeSet &Set, Module *srcM, unsigned mode)<br>
  Â  Â  Â  Â : DstM(dstM), SrcM(srcM), TypeMap(Set),<br>
-  Â  Â  Â  Â  ValMaterializer(TypeMap, DstM, LazilyLinkFunctions), Mode(mode),<br>
-  Â  Â  Â  Â  SuppressWarnings(SuppressWarnings) {}<br>
+  Â  Â  Â  Â  ValMaterializer(TypeMap, DstM, LazilyLinkFunctions), Mode(mode) {}<br>
<br>
  Â  Â bool run();<br>
<br>
  Â private:<br>
-  Â  bool shouldLinkFromSource(const GlobalValue &Dest, const GlobalValue &Src);<br>
+  Â  bool shouldLinkFromSource(bool &LinkFromSrc, const GlobalValue &Dest,<br>
+  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  const GlobalValue &Src);<br>
<br>
-  Â  /// emitError - Helper method for setting a message and returning an error<br>
-  Â  /// code.<br>
+  Â  /// Helper method for setting a message and returning an error code.<br>
  Â  Â bool emitError(const Twine &Message) {<br>
-  Â  Â  ErrorMsg = Message.str();<br>
+  Â  Â  DstM->getContext().diagnose(LinkDiagnosticInfo(DS_Error, Message));<br>
  Â  Â  Â return true;<br>
  Â  Â }<br>
<br>
+  Â  void emitWarning(const Twine &Message) {<br>
+  Â  Â  DstM->getContext().diagnose(LinkDiagnosticInfo(DS_Warning, Message));<br>
+  Â  }<br>
+<br>
  Â  Â bool getComdatLeader(Module *M, StringRef ComdatName,<br>
  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  const GlobalVariable *&GVar);<br>
  Â  Â bool computeResultingSelectionKind(StringRef ComdatName,<br>
@@ -672,7 +687,8 @@ bool ModuleLinker::getComdatResult(const<br>
  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  LinkFromSrc);<br>
 }<br>
<br>
-bool ModuleLinker::shouldLinkFromSource(const GlobalValue &Dest,<br>
+bool ModuleLinker::shouldLinkFromSource(bool &LinkFromSrc,<br>
+  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  const GlobalValue &Dest,<br>
  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â const GlobalValue &Src) {<br>
  Â bool SrcIsDeclaration = Src.isDeclarationForLinker();<br>
  Â bool DestIsDeclaration = Dest.isDeclarationForLinker();<br>
@@ -683,42 +699,56 @@ bool ModuleLinker::shouldLinkFromSource(<br>
  Â if (SrcIsDeclaration) {<br>
  Â  Â // If Src is external or if both Src & Dest are external..  Just link the<br>
  Â  Â // external globals, we aren't adding anything.<br>
-  Â  if (Src.hasDLLImportStorageClass())<br>
+  Â  if (Src.hasDLLImportStorageClass()) {<br>
  Â  Â  Â // If one of GVs is marked as DLLImport, result should be dllimport'ed.<br>
-  Â  Â  return DestIsDeclaration;<br>
+  Â  Â  LinkFromSrc = DestIsDeclaration;<br>
+  Â  Â  return false;<br>
+  Â  }<br>
  Â  Â // If the Dest is weak, use the source linkage.<br>
-  Â  return Dest.hasExternalWeakLinkage();<br>
+  Â  LinkFromSrc = Dest.hasExternalWeakLinkage();<br>
+  Â  return false;<br>
  Â }<br>
<br>
-  if (DestIsDeclaration)<br>
+  if (DestIsDeclaration) {<br>
  Â  Â // If Dest is external but Src is not:<br>
-  Â  return true;<br>
+  Â  LinkFromSrc = true;<br>
+  Â  return false;<br>
+  }<br>
<br>
  Â if (Src.hasCommonLinkage()) {<br>
-  Â  if (Dest.hasLinkOnceLinkage() || Dest.hasWeakLinkage())<br>
-  Â  Â  return true;<br>
+  Â  if (Dest.hasLinkOnceLinkage() || Dest.hasWeakLinkage()) {<br>
+  Â  Â  LinkFromSrc = true;<br>
+  Â  Â  return false;<br>
+  Â  }<br>
<br>
-  Â  if (!Dest.hasCommonLinkage())<br>
+  Â  if (!Dest.hasCommonLinkage()) {<br>
+  Â  Â  LinkFromSrc = false;<br>
  Â  Â  Â return false;<br>
+  Â  }<br>
<br>
  Â  Â uint64_t DestSize = DL.getTypeAllocSize(Dest.getType()->getElementType());<br>
  Â  Â uint64_t SrcSize = DL.getTypeAllocSize(Src.getType()->getElementType());<br>
-  Â  return SrcSize > DestSize;<br>
+  Â  LinkFromSrc = SrcSize > DestSize;<br>
+  Â  return false;<br>
  Â }<br>
<br>
  Â if (Src.isWeakForLinker()) {<br>
  Â  Â assert(!Dest.hasExternalWeakLinkage());<br>
  Â  Â assert(!Dest.hasAvailableExternallyLinkage());<br>
<br>
-  Â  if (Dest.hasLinkOnceLinkage() && Src.hasWeakLinkage())<br>
-  Â  Â  return true;<br>
+  Â  if (Dest.hasLinkOnceLinkage() && Src.hasWeakLinkage()) {<br>
+  Â  Â  LinkFromSrc = true;<br>
+  Â  Â  return false;<br>
+  Â  }<br>
<br>
+  Â  LinkFromSrc = false;<br>
  Â  Â return false;<br>
  Â }<br>
<br>
  Â if (Dest.isWeakForLinker()) {<br>
  Â  Â assert(Src.hasExternalLinkage());<br>
-  Â  return true;<br>
+  Â  LinkFromSrc = true;<br>
+  Â  return false;<br>
  Â }<br>
<br>
  Â assert(!Src.hasExternalWeakLinkage());<br>
@@ -742,9 +772,7 @@ bool ModuleLinker::getLinkageResult(Glob<br>
  Â assert(!Src->hasLocalLinkage() &&<br>
  Â  Â  Â  Â  "If Src has internal linkage, Dest shouldn't be set!");<br>
<br>
-  assert(ErrorMsg.empty());<br>
-  LinkFromSrc = shouldLinkFromSource(*Dest, *Src);<br>
-  if (!ErrorMsg.empty())<br>
+  if (shouldLinkFromSource(LinkFromSrc, *Dest, *Src))<br>
  Â  Â return true;<br>
<br>
  Â if (LinkFromSrc)<br>
@@ -1470,10 +1498,8 @@ bool ModuleLinker::linkModuleFlagsMetada<br>
  Â  Â case Module::Warning: {<br>
  Â  Â  Â // Emit a warning if the values differ.<br>
  Â  Â  Â if (SrcOp->getOperand(2) != DstOp->getOperand(2)) {<br>
-  Â  Â  Â  if (!SuppressWarnings) {<br>
-  Â  Â  Â  Â  errs() << "WARNING: linking module flags '" << ID->getString()<br>
-  Â  Â  Â  Â  Â  Â  Â  Â << "': IDs have conflicting values";<br>
-  Â  Â  Â  }<br>
+  Â  Â  Â  emitWarning("linking module flags '" + ID->getString() +<br>
+  Â  Â  Â  Â  Â  Â  Â  Â  Â  "': IDs have conflicting values");<br>
  Â  Â  Â }<br>
  Â  Â  Â continue;<br>
  Â  Â }<br>
@@ -1540,23 +1566,19 @@ bool ModuleLinker::run() {<br>
<br>
  Â if (SrcM->getDataLayout() && DstM->getDataLayout() &&<br>
  Â  Â  Â *SrcM->getDataLayout() != *DstM->getDataLayout()) {<br>
-  Â  if (!SuppressWarnings) {<br>
-  Â  Â  errs() << "WARNING: Linking two modules of different data layouts: '"<br>
-  Â  Â  Â  Â  Â  Â << SrcM->getModuleIdentifier() << "' is '"<br>
-  Â  Â  Â  Â  Â  Â << SrcM->getDataLayoutStr() << "' whereas '"<br>
-  Â  Â  Â  Â  Â  Â << DstM->getModuleIdentifier() << "' is '"<br>
-  Â  Â  Â  Â  Â  Â << DstM->getDataLayoutStr() << "'\n";<br>
-  Â  }<br>
+  Â  emitWarning("Linking two modules of different data layouts: '" +<br>
+  Â  Â  Â  Â  Â  Â  Â  SrcM->getModuleIdentifier() + "' is '" +<br>
+  Â  Â  Â  Â  Â  Â  Â  SrcM->getDataLayoutStr() + "' whereas '" +<br>
+  Â  Â  Â  Â  Â  Â  Â  DstM->getModuleIdentifier() + "' is '" +<br>
+  Â  Â  Â  Â  Â  Â  Â  DstM->getDataLayoutStr() + "'\n");<br>
  Â }<br>
  Â if (!SrcM->getTargetTriple().empty() &&<br>
  Â  Â  Â DstM->getTargetTriple() != SrcM->getTargetTriple()) {<br>
-  Â  if (!SuppressWarnings) {<br>
-  Â  Â  errs() << "WARNING: Linking two modules of different target triples: "<br>
-  Â  Â  Â  Â  Â  Â << SrcM->getModuleIdentifier() << "' is '"<br>
-  Â  Â  Â  Â  Â  Â << SrcM->getTargetTriple() << "' whereas '"<br>
-  Â  Â  Â  Â  Â  Â << DstM->getModuleIdentifier() << "' is '"<br>
-  Â  Â  Â  Â  Â  Â << DstM->getTargetTriple() << "'\n";<br>
-  Â  }<br>
+  Â  emitWarning("Linking two modules of different target triples: " +<br>
+  Â  Â  Â  Â  Â  Â  Â  SrcM->getModuleIdentifier() + "' is '" +<br>
+  Â  Â  Â  Â  Â  Â  Â  SrcM->getTargetTriple() + "' whereas '" +<br>
+  Â  Â  Â  Â  Â  Â  Â  DstM->getModuleIdentifier() + "' is '" +<br>
+  Â  Â  Â  Â  Â  Â  Â  DstM->getTargetTriple() + "'\n");<br>
  Â }<br>
<br>
  Â // Append the module inline asm string.<br>
@@ -1626,10 +1648,8 @@ bool ModuleLinker::run() {<br>
<br>
  Â  Â // Materialize if needed.<br>
  Â  Â if (SF->isMaterializable()) {<br>
-  Â  Â  if (std::error_code EC = SF->materialize()) {<br>
-  Â  Â  Â  ErrorMsg = EC.message();<br>
-  Â  Â  Â  return true;<br>
-  Â  Â  }<br>
+  Â  Â  if (std::error_code EC = SF->materialize())<br>
+  Â  Â  Â  return emitError(EC.message());<br>
  Â  Â }<br>
<br>
  Â  Â // Skip if no body (function is external).<br>
@@ -1679,10 +1699,8 @@ bool ModuleLinker::run() {<br>
<br>
  Â  Â  Â // Materialize if needed.<br>
  Â  Â  Â if (SF->isMaterializable()) {<br>
-  Â  Â  Â  if (std::error_code EC = SF->materialize()) {<br>
-  Â  Â  Â  Â  ErrorMsg = EC.message();<br>
-  Â  Â  Â  Â  return true;<br>
-  Â  Â  Â  }<br>
+  Â  Â  Â  if (std::error_code EC = SF->materialize())<br>
+  Â  Â  Â  Â  return emitError(EC.message());<br>
  Â  Â  Â }<br>
<br>
  Â  Â  Â // Skip if no body (function is external).<br>
@@ -1711,8 +1729,7 @@ bool ModuleLinker::run() {<br>
  Â return false;<br>
 }<br>
<br>
-Linker::Linker(Module *M, bool SuppressWarnings)<br>
-  Â  : Composite(M), SuppressWarnings(SuppressWarnings) {<br>
+Linker::Linker(Module *M) : Composite(M) {<br>
  Â TypeFinder StructTypes;<br>
  Â StructTypes.run(*M, true);<br>
  Â IdentifiedStructTypes.insert(StructTypes.begin(), StructTypes.end());<br>
@@ -1726,15 +1743,9 @@ void Linker::deleteModule() {<br>
  Â Composite = nullptr;<br>
 }<br>
<br>
-bool Linker::linkInModule(Module *Src, unsigned Mode, std::string *ErrorMsg) {<br>
-  ModuleLinker TheLinker(Composite, IdentifiedStructTypes, Src, Mode,<br>
-  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â SuppressWarnings);<br>
-  if (TheLinker.run()) {<br>
-  Â  if (ErrorMsg)<br>
-  Â  Â  *ErrorMsg = TheLinker.ErrorMsg;<br>
-  Â  return true;<br>
-  }<br>
-  return false;<br>
+bool Linker::linkInModule(Module *Src, unsigned Mode) {<br>
+  ModuleLinker TheLinker(Composite, IdentifiedStructTypes, Src, Mode);<br>
+  return TheLinker.run();<br>
 }<br>
<br>
 //===----------------------------------------------------------------------===//<br>
@@ -1746,10 +1757,9 @@ bool Linker::linkInModule(Module *Src, u<br>
 /// error occurs, true is returned and ErrorMsg (if not null) is set to indicate<br>
 /// the problem.  Upon failure, the Dest module could be in a modified state,<br>
 /// and shouldn't be relied on to be consistent.<br>
-bool Linker::LinkModules(Module *Dest, Module *Src, unsigned Mode,<br>
-  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â std::string *ErrorMsg) {<br>
+bool Linker::LinkModules(Module *Dest, Module *Src, unsigned Mode) {<br>
  Â Linker L(Dest);<br>
-  return L.linkInModule(Src, Mode, ErrorMsg);<br>
+  return L.linkInModule(Src, Mode);<br>
 }<br>
<br>
 //===----------------------------------------------------------------------===//<br>
@@ -1758,10 +1768,6 @@ bool Linker::LinkModules(Module *Dest, M<br>
<br>
 LLVMBool LLVMLinkModules(LLVMModuleRef Dest, LLVMModuleRef Src,<br>
  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  LLVMLinkerMode Mode, char **OutMessages) {<br>
-  std::string Messages;<br>
-  LLVMBool Result = Linker::LinkModules(unwrap(Dest), unwrap(Src),<br>
-  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Mode, OutMessages? &Messages : nullptr);<br>
-  if (OutMessages)<br>
-  Â  *OutMessages = strdup(Messages.c_str());<br>
+  LLVMBool Result = Linker::LinkModules(unwrap(Dest), unwrap(Src), Mode);<br>
  Â return Result;<br>
 }<br>
<br>
Modified: llvm/trunk/tools/bugpoint/BugDriver.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/bugpoint/BugDriver.cpp?rev=220608&r1=220607&r2=220608&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/bugpoint/BugDriver.cpp?rev=220608&r1=220607&r2=220608&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/tools/bugpoint/BugDriver.cpp (original)<br>
+++ llvm/trunk/tools/bugpoint/BugDriver.cpp Fri Oct 24 23:06:10 2014<br>
@@ -126,13 +126,8 @@ bool BugDriver::addSources(const std::ve<br>
  Â  Â if (!M.get()) return true;<br>
<br>
  Â  Â outs() << "Linking in input file: '" << Filenames[i] << "'\n";<br>
-  Â  std::string ErrorMessage;<br>
-  Â  if (Linker::LinkModules(Program, M.get(), Linker::DestroySource,<br>
-  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  &ErrorMessage)) {<br>
-  Â  Â  errs() << ToolName << ": error linking in '" << Filenames[i] << "': "<br>
-  Â  Â  Â  Â  Â  Â << ErrorMessage << '\n';<br>
+  Â  if (Linker::LinkModules(Program, M.get(), Linker::DestroySource))<br>
  Â  Â  Â return true;<br>
-  Â  }<br>
  Â }<br>
<br>
  Â outs() << "*** All input ok\n";<br>
<br>
Modified: llvm/trunk/tools/bugpoint/Miscompilation.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/bugpoint/Miscompilation.cpp?rev=220608&r1=220607&r2=220608&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/bugpoint/Miscompilation.cpp?rev=220608&r1=220607&r2=220608&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/tools/bugpoint/Miscompilation.cpp (original)<br>
+++ llvm/trunk/tools/bugpoint/Miscompilation.cpp Fri Oct 24 23:06:10 2014<br>
@@ -218,16 +218,12 @@ static Module *TestMergedProgram(const B<br>
  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  bool DeleteInputs, std::string &Error,<br>
  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  bool &Broken) {<br>
  Â // Link the two portions of the program back to together.<br>
-  std::string ErrorMsg;<br>
  Â if (!DeleteInputs) {<br>
  Â  Â M1 = CloneModule(M1);<br>
  Â  Â M2 = CloneModule(M2);<br>
  Â }<br>
-  if (Linker::LinkModules(M1, M2, Linker::DestroySource, &ErrorMsg)) {<br>
-  Â  errs() << BD.getToolName() << ": Error linking modules together:"<br>
-  Â  Â  Â  Â  Â << ErrorMsg << '\n';<br>
+  if (Linker::LinkModules(M1, M2, Linker::DestroySource))<br>
  Â  Â exit(1);<br>
-  }<br>
  Â delete M2;  Â // We are done with this module.<br>
<br>
  Â // Execute the program.<br>
@@ -396,13 +392,9 @@ static bool ExtractLoops(BugDriver &BD,<br>
  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â F->getFunctionType()));<br>
  Â  Â  Â }<br>
<br>
-  Â  Â  std::string ErrorMsg;<br>
-  Â  Â  if (Linker::LinkModules(ToNotOptimize, ToOptimizeLoopExtracted,<br>
-  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Linker::DestroySource, &ErrorMsg)){<br>
-  Â  Â  Â  errs() << BD.getToolName() << ": Error linking modules together:"<br>
-  Â  Â  Â  Â  Â  Â  Â << ErrorMsg << '\n';<br>
+  Â  Â  if (Linker::LinkModules(ToNotOptimize, ToOptimizeLoopExtracted,<br>
+  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Linker::DestroySource))<br>
  Â  Â  Â  Â exit(1);<br>
-  Â  Â  }<br>
<br>
  Â  Â  Â MiscompiledFunctions.clear();<br>
  Â  Â  Â for (unsigned i = 0, e = MisCompFunctions.size(); i != e; ++i) {<br>
@@ -430,13 +422,10 @@ static bool ExtractLoops(BugDriver &BD,<br>
  Â  Â // extraction both didn't break the program, and didn't mask the problem.<br>
  Â  Â // Replace the current program with the loop extracted version, and try to<br>
  Â  Â // extract another loop.<br>
-  Â  std::string ErrorMsg;<br>
-  Â  if (Linker::LinkModules(ToNotOptimize, ToOptimizeLoopExtracted,<br>
-  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Linker::DestroySource, &ErrorMsg)){<br>
-  Â  Â  errs() << BD.getToolName() << ": Error linking modules together:"<br>
-  Â  Â  Â  Â  Â  Â << ErrorMsg << '\n';<br>
+  Â  if (Linker::LinkModules(ToNotOptimize, ToOptimizeLoopExtracted,<br>
+  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Linker::DestroySource))<br>
  Â  Â  Â exit(1);<br>
-  Â  }<br>
+<br>
  Â  Â delete ToOptimizeLoopExtracted;<br>
<br>
  Â  Â // All of the Function*'s in the MiscompiledFunctions list are in the old<br>
@@ -612,13 +601,8 @@ static bool ExtractBlocks(BugDriver &BD,<br>
  Â  Â  Â MisCompFunctions.push_back(std::make_pair(I->getName(),<br>
  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â I->getFunctionType()));<br>
<br>
-  std::string ErrorMsg;<br>
-  if (Linker::LinkModules(ProgClone, Extracted.get(), Linker::DestroySource,<br>
-  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  &ErrorMsg)) {<br>
-  Â  errs() << BD.getToolName() << ": Error linking modules together:"<br>
-  Â  Â  Â  Â  Â << ErrorMsg << '\n';<br>
+  if (Linker::LinkModules(ProgClone, Extracted.get(), Linker::DestroySource))<br>
  Â  Â exit(1);<br>
-  }<br>
<br>
  Â // Set the new program and delete the old one.<br>
  Â BD.setNewProgram(ProgClone);<br>
<br>
Modified: llvm/trunk/tools/gold/gold-plugin.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/gold/gold-plugin.cpp?rev=220608&r1=220607&r2=220608&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/gold/gold-plugin.cpp?rev=220608&r1=220607&r2=220608&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/tools/gold/gold-plugin.cpp (original)<br>
+++ llvm/trunk/tools/gold/gold-plugin.cpp Fri Oct 24 23:06:10 2014<br>
@@ -777,9 +777,8 @@ static ld_plugin_status allSymbolsReadHo<br>
  Â  Â  Â M->setTargetTriple(DefaultTriple);<br>
  Â  Â }<br>
<br>
-  Â  std::string ErrMsg;<br>
-  Â  if (L.linkInModule(M.get(), &ErrMsg))<br>
-  Â  Â  message(LDPL_FATAL, "Failed to link module: %s", ErrMsg.c_str());<br>
+  Â  if (L.linkInModule(M.get()))<br>
+  Â  Â  message(LDPL_FATAL, "Failed to link module");<br>
  Â }<br>
<br>
  Â for (const auto &Name : Internalize) {<br>
<br>
Modified: llvm/trunk/tools/llvm-link/llvm-link.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-link/llvm-link.cpp?rev=220608&r1=220607&r2=220608&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-link/llvm-link.cpp?rev=220608&r1=220607&r2=220608&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/tools/llvm-link/llvm-link.cpp (original)<br>
+++ llvm/trunk/tools/llvm-link/llvm-link.cpp Fri Oct 24 23:06:10 2014<br>
@@ -14,6 +14,8 @@<br>
<br>
 #include "llvm/Linker/Linker.h"<br>
 #include "llvm/Bitcode/ReaderWriter.h"<br>
+#include "llvm/IR/DiagnosticInfo.h"<br>
+#include "llvm/IR/DiagnosticPrinter.h"<br>
 #include "llvm/IR/LLVMContext.h"<br>
 #include "llvm/IR/Module.h"<br>
 #include "llvm/IR/Verifier.h"<br>
@@ -69,6 +71,25 @@ loadFile(const char *argv0, const std::s<br>
  Â return Result;<br>
 }<br>
<br>
+static void diagnosticHandler(const DiagnosticInfo &DI, void *Context) {<br>
+  unsigned Severity = DI.getSeverity();<br>
+  switch (Severity) {<br>
+  case DS_Error:<br>
+  Â  errs() << "ERROR: ";<br>
+  case DS_Warning:<br>
+  Â  if (SuppressWarnings)<br>
+  Â  Â  return;<br>
+  Â  errs() << "WARNING: ";<br>
+  Â  break;<br>
+  case DS_Remark:<br>
+  case DS_Note:<br>
+  Â  llvm_unreachable("Only expecting warnings and errors");<br>
+  }<br>
+<br>
+  DiagnosticPrinterRawOStream DP(errs());<br>
+  DI.print(DP);<br>
+}<br>
+<br>
 int main(int argc, char **argv) {<br>
  Â // Print a stack trace if we signal out.<br>
  Â sys::PrintStackTraceOnErrorSignal();<br>
@@ -79,9 +100,9 @@ int main(int argc, char **argv) {<br>
  Â cl::ParseCommandLineOptions(argc, argv, "llvm linker\n");<br>
<br>
  Â auto Composite = make_unique<Module>("llvm-link", Context);<br>
-  Linker L(Composite.get(), SuppressWarnings);<br>
+  Linker L(Composite.get());<br>
<br>
-  std::string ErrorMessage;<br>
+  Context.setDiagnosticHandler(diagnosticHandler);<br>
  Â for (unsigned i = 0; i < InputFilenames.size(); ++i) {<br>
  Â  Â std::unique_ptr<Module> M = loadFile(argv[0], InputFilenames[i], Context);<br>
  Â  Â if (!M.get()) {<br>
@@ -91,11 +112,8 @@ int main(int argc, char **argv) {<br>
<br>
  Â  Â if (Verbose) errs() << "Linking in '" << InputFilenames[i] << "'\n";<br>
<br>
-  Â  if (L.linkInModule(M.get(), &ErrorMessage)) {<br>
-  Â  Â  errs() << argv[0] << ": link error in '" << InputFilenames[i]<br>
-  Â  Â  Â  Â  Â  Â << "': " << ErrorMessage << "\n";<br>
+  Â  if (L.linkInModule(M.get()))<br>
  Â  Â  Â return 1;<br>
-  Â  }<br>
  Â }<br>
<br>
  Â if (DumpAsm) errs() << "Here's the assembly:\n" << *Composite;<br>
<br>
Modified: llvm/trunk/tools/llvm-lto/llvm-lto.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-lto/llvm-lto.cpp?rev=220608&r1=220607&r2=220608&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-lto/llvm-lto.cpp?rev=220608&r1=220607&r2=220608&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/tools/llvm-lto/llvm-lto.cpp (original)<br>
+++ llvm/trunk/tools/llvm-lto/llvm-lto.cpp Fri Oct 24 23:06:10 2014<br>
@@ -143,12 +143,8 @@ int main(int argc, char **argv) {<br>
  Â  Â  Â return 1;<br>
  Â  Â }<br>
<br>
-<br>
-  Â  if (!CodeGen.addModule(Module.get(), error)) {<br>
-  Â  Â  errs() << argv[0] << ": error adding file '" << InputFilenames[i]<br>
-  Â  Â  Â  Â  Â  Â << "': " << error << "\n";<br>
+  Â  if (!CodeGen.addModule(Module.get()))<br>
  Â  Â  Â return 1;<br>
-  Â  }<br>
<br>
  Â  Â unsigned NumSyms = Module->getSymbolCount();<br>
  Â  Â for (unsigned I = 0; I < NumSyms; ++I) {<br>
<br>
Modified: llvm/trunk/tools/lto/lto.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/lto/lto.cpp?rev=220608&r1=220607&r2=220608&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/lto/lto.cpp?rev=220608&r1=220607&r2=220608&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/tools/lto/lto.cpp (original)<br>
+++ llvm/trunk/tools/lto/lto.cpp Fri Oct 24 23:06:10 2014<br>
@@ -205,7 +205,7 @@ lto_code_gen_t lto_codegen_create(void)<br>
 void lto_codegen_dispose(lto_code_gen_t cg) { delete unwrap(cg); }<br>
<br>
 bool lto_codegen_add_module(lto_code_gen_t cg, lto_module_t mod) {<br>
-  return !unwrap(cg)->addModule(unwrap(mod), sLastErrorString);<br>
+  return !unwrap(cg)->addModule(unwrap(mod));<br>
 }<br>
<br>
 bool lto_codegen_set_debug_model(lto_code_gen_t cg, lto_debug_model debug) {<br>
<br>
Modified: llvm/trunk/unittests/Linker/LinkModulesTest.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/unittests/Linker/LinkModulesTest.cpp?rev=220608&r1=220607&r2=220608&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/unittests/Linker/LinkModulesTest.cpp?rev=220608&r1=220607&r2=220608&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/unittests/Linker/LinkModulesTest.cpp (original)<br>
+++ llvm/trunk/unittests/Linker/LinkModulesTest.cpp Fri Oct 24 23:06:10 2014<br>
@@ -88,7 +88,7 @@ TEST_F(LinkModuleTest, BlockAddress) {<br>
  Â Builder.CreateRet(ConstantPointerNull::get(Type::getInt8PtrTy(Ctx)));<br>
<br>
  Â Module *LinkedModule = new Module("MyModuleLinked", Ctx);<br>
-  Linker::LinkModules(LinkedModule, M.get(), Linker::PreserveSource, nullptr);<br>
+  Linker::LinkModules(LinkedModule, M.get(), Linker::PreserveSource);<br>
<br>
  Â // Delete the original module.<br>
  Â M.reset();<br>
@@ -143,11 +143,11 @@ TEST_F(LinkModuleTest, EmptyModule) {<br>
  Â GV->setInitializer(ConstantStruct::get(STy, F));<br>
<br>
  Â Module *EmptyM = new Module("EmptyModule1", Ctx);<br>
-  Linker::LinkModules(EmptyM, InternalM, Linker::PreserveSource, nullptr);<br>
+  Linker::LinkModules(EmptyM, InternalM, Linker::PreserveSource);<br>
<br>
  Â delete EmptyM;<br>
  Â EmptyM = new Module("EmptyModule2", Ctx);<br>
-  Linker::LinkModules(InternalM, EmptyM, Linker::PreserveSource, nullptr);<br>
+  Linker::LinkModules(InternalM, EmptyM, Linker::PreserveSource);<br>
<br>
  Â delete EmptyM;<br>
  Â delete InternalM;<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</blockquote></div><div><br></div>
</div></div></div></div>
</blockquote></div><br><br>
</div></div></div>