[PATCH] D50560: [LLD] Enable Visual Studio compatible diagnostics.
    Igor Kudrin via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Thu Sep 20 19:00:16 PDT 2018
    
    
  
ikudrin added a comment.
With overloaded operators we can write something like
  error("relocation " + toString(Type) +
          " cannot refer to absolute symbol: " + toString(Sym) +
          definedAt(toObjLoc(Sym.File)) +
          referencedBy(S.getSymLoc(Sym, RelOff)));
How would you prefer it to look like?
I don't insist on the current style, but I believe it is important to be discussed prior to rewriting the code because the decision will influence the design directly.
Repository:
  rLLD LLVM Linker
https://reviews.llvm.org/D50560
    
    
More information about the llvm-commits
mailing list