<div dir="ltr">Hi guys,<div><br></div><div>I submitted <a href="https://reviews.llvm.org/rL299097">https://reviews.llvm.org/rL299097</a> and <a href="https://reviews.llvm.org/rLr299280">https://reviews.llvm.org/rLr299280</a> to change the error message styles for undefined or duplicated symbols.</div><div><br></div><div>I want to get stable error message formats until the next LLVM release, but until then, we can change them freely. So you don't need to think that they are in the final form. I'll try to use this to see if further changes are needed. If you are interested, please do that yourself too.</div><div><br></div><div>Thanks,</div><div>Rui</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 29, 2017 at 3:28 PM, Rui Ueyama <span dir="ltr"><<a href="mailto:ruiu@google.com" target="_blank">ruiu@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">The first patch is sent out: <a href="https://reviews.llvm.org/D31481" target="_blank">https://reviews.llvm.org/<wbr>D31481</a><br></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 29, 2017 at 1:47 PM, Rafael Espíndola <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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I am late on the thread, but I just want to say that the new format<br>
looks awesome!<br>
<br>
Thanks,<br>
Rafael<br>
<br>
<br>
On 29 March 2017 at 15:18, Rui Ueyama via llvm-dev<br>
<div class="m_-4896683389129853639HOEnZb"><div class="m_-4896683389129853639h5"><<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br>
> My bad. I intended this.<br>
><br>
> Undefined symbol error:<br>
><br>
> bin/ld.lld: error: undefined symbol:<br>
> lld::elf::EhFrameSection<llvm:<wbr>:object::ELFType<(llvm::suppor<wbr>t::endianness)0,<br>
> true><br>
>>>> Referenced by Writer.cpp:207 (/ssd/llvm-project/lld/ELF/Wri<wbr>ter.cpp:207)<br>
>>>> Writer.cpp.o in archive lib/liblldELF.a<br>
><br>
><br>
> Duplicate symbol error:<br>
><br>
> bin/ld.lld: error: duplicate symbol:<br>
> lld::elf::MipsGotSection::addE<wbr>ntry(lld::elf::SymbolBody&, long,<br>
> lld::elf::RelExpr)<br>
>>>> defined at Writer.cpp:38<br>
>>>> (/home/buildslave/buildslave/c<wbr>lang-cmake-aarch64-39vma/llvm/<wbr>tools/lld/ELF/Writer.cpp:38)<br>
>>>> Writer.cpp.o in archive lib/liblldELF.a<br>
>>>> defined at SyntheticSections.cpp:673<br>
>>>> (/home/buildslave/buildslave/c<wbr>lang-cmake-aarch64-39vma/llvm/<wbr>tools/lld/ELF/SyntheticSection<wbr>s.cpp:673)<br>
>>>> SyntheticSections.cpp.o in archive lib/liblldELF.a<br>
><br>
><br>
> On Wed, Mar 29, 2017 at 12:12 PM, Jonathan Roelofs<br>
> <<a href="mailto:jonathan@codesourcery.com" target="_blank">jonathan@codesourcery.com</a>> wrote:<br>
>><br>
>><br>
>><br>
>> On 3/29/17 12:53 PM, Rui Ueyama via llvm-dev wrote:<br>
>>><br>
>>> Put it all together, the following error messages should work for<br>
>>> everybody. I'll create a patch to make this change and send it for<br>
>>> review. Thank you guys for the inputs!<br>
>>><br>
>>><br>
>>> Undefined symbol error:<br>
>>><br>
>>> bin/ld.lld: error: undefined symbol:<br>
>>><br>
>>> lld::elf::EhFrameSection<llvm:<wbr>:object::ELFType<(llvm::suppor<wbr>t::endianness)0,<br>
>>> true><br>
>>> *>>> defined at* Writer.cpp:207<br>
>>> (/ssd/llvm-project/lld/ELF/Wri<wbr>ter.cpp:207)<br>
>>> *>>>* Writer.cpp.o in archive lib/liblldELF.a<br>
>><br>
>><br>
>> The wording of this one is mildly self contradictory. Undefined symbols<br>
>> aren't defined, they're... used? referenced?<br>
>><br>
>><br>
>> Jon<br>
>><br>
>>><br>
>>><br>
>>> Duplicate symbol error:<br>
>>><br>
>>> bin/ld.lld: error: duplicate symbol:<br>
>>> lld::elf::MipsGotSection::addE<wbr>ntry(lld::elf::SymbolBody&, long,<br>
>>> lld::elf::RelExpr)<br>
>>> *>>> defined at* Writer.cpp:38<br>
>>><br>
>>> (/home/buildslave/buildslave/c<wbr>lang-cmake-aarch64-39vma/llvm/<wbr>tools/lld/ELF/Writer.cpp:38)<br>
>>> *>>>* Writer.cpp.o in archive lib/liblldELF.a<br>
>>> *>>> defined at* SyntheticSections.cpp:673<br>
>>><br>
>>> (/home/buildslave/buildslave/c<wbr>lang-cmake-aarch64-39vma/llvm/<wbr>tools/lld/ELF/SyntheticSection<wbr>s.cpp:673)<br>
>>> *>>>* SyntheticSections.cpp.o in archive lib/liblldELF.a<br>
>>><br>
>>><br>
>><br>
>><br>
>> --<br>
>> Jon Roelofs<br>
>> <a href="mailto:jonathan@codesourcery.com" target="_blank">jonathan@codesourcery.com</a><br>
>> CodeSourcery / Mentor Embedded<br>
><br>
><br>
><br>
</div></div><div class="m_-4896683389129853639HOEnZb"><div class="m_-4896683389129853639h5">> ______________________________<wbr>_________________<br>
> LLVM Developers mailing list<br>
> <a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-dev</a><br>
><br>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>