[PATCH] D27676: [ELF] - Use full object name if source file name exist when reporting errors.
    Sean Silva via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Tue Dec 13 20:51:09 PST 2016
    
    
  
silvas added a comment.
In https://reviews.llvm.org/D27676#621721, @ruiu wrote:
> I think my concern is that we are going to print out at most three filenames like this
>
>   foo.a(foo.o)(foo.c)
>   
>
> and that error string format looks odd. It is indistinguishable from
>
>   foo.o(foo.c)
>   
>
> where in this case foo.o is not an archive but an object file. I think we are using too many parentheses.
In the post-commit of r285186 I suggested using a `note:` to report the object file.
Maybe when we use a source location for the error, we can add a note something like:
  note: this definition was found in baz.a(bar.o)
https://reviews.llvm.org/D27676
    
    
More information about the llvm-commits
mailing list