<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Feb 6, 2018 at 3:11 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 <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> writes:<br>
<br>
> ruiu added a comment.<br>
><br>
> I don't think I like this patch because it makes things more complicated than it can be. "Supported targets" are just informative for compatibility with GNU linkers, and it doesn't change often. I'm more comfortable with the current code than this one.<br>
<br>
</span>Can we trim the output then?<br>
<br>
Currently we have<br>
<br>
-  outs() << Argv0 << ": supported targets: "<br>
-         << "elf32-i386 elf32-iamcu elf32-littlearm elf32-ntradbigmips "<br>
-         << "elf32-ntradlittlemips elf32-powerpc elf32-tradbigmips "<br>
-         << "elf32-tradlittlemips elf32-x86-64 "<br>
-         << "elf64-amdgpu elf64-littleaarch64 elf64-powerpc elf64-tradbigmips "<br>
-         << "elf64-tradlittlemips elf64-x86-64\n";<br>
<br>
Could we print something like "elf32-* elf64-*"?<br></blockquote><div><br></div><div>It was added to appease libtool, and I believe libtool expects /supported targets:.* elf/. So "supported targets: elf" should be enough.</div></div></div></div>