[PATCH] D31208: Print out "suppoted targets" and "supported emulations".

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 22 10:26:07 PDT 2017


Libtool expects "targets", so that would result in "supported targets:
elf", which is odd. So I prefer this style.

On the other hand I can remove "supported emulations" completely because I
think Libtool doesn't need it.

On Wed, Mar 22, 2017 at 6:03 AM, Rafael Avila de Espindola <
rafael.espindola at gmail.com> wrote:

> Rui Ueyama via Phabricator <reviews at reviews.llvm.org> writes:
>
> > ruiu added a comment.
> >
> > I chose to print out all the targets and emulations that LLD actually
> supports, instead of printing out `supported target: elf`, because I found
> that the message is useful not only for Libtool-generated scripts but for
> human as well.
>
> There are very few cases when a -m option is needed, and I don't thing
> the "supported target" list adds any value to the user, so I would
> prefer just "supported target: elf".
>
> Cheers,
> Rafael
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170322/90ae1bde/attachment.html>


More information about the llvm-commits mailing list