<div dir="ltr">For my particular use case, only the names and values matter.  But I think it's just bad form to implement a feature for compatibility with GNU linkers and then gratuitously fail to make it actually compatible just because of laziness.</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Dec 1, 2017 at 6:02 PM, Rafael Avila de 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">Rui Ueyama via Phabricator via llvm-commits<br>
<<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> writes:<br>
<br>
</span><span class="">> ruiu marked 5 inline comments as done.<br>
> ruiu added inline comments.<br>
><br>
><br>
> ================<br>
> Comment at: lld/ELF/InputFiles.cpp:1055<br>
> +template <class ELFT><br>
> +std::vector<std::pair<<wbr>StringRef, uint64_t>><br>
> +elf::readSymbols(<wbr>MemoryBufferRef MB) {<br>
> ----------------<br>
> mcgrathr wrote:<br>
>> The canonical semantics are to copy all the other fields as well, not just st_value.<br>
>> Only st_shndx is replaced with SHN_ABS.  The others are preserved (size, visibility, type, etc.).<br>
>><br>
> Yeah that's true, but I wonder if real programs depend on it.<br>
<br>
</span>Visibility might have an impact in what relaxations get applied, but<br>
that does sound fairly contrived.<br>
<br>
Roland, do you have a use case in mind for anything but the names?<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div>