[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