[cfe-commits] r154389 - in /cfe/trunk: lib/Driver/Tools.cpp test/Driver/linux-as.c

[Chandler Carruth]

On Mon, Apr 23, 2012 at 12:14 AM, Evgeniy Stepanov <
eugeni.stepanov at gmail.com> wrote:

> On Sun, Apr 22, 2012 at 5:41 AM, Hal Finkel <hfinkel at anl.gov> wrote:
> > Evgeniy,
> >
> > Unfortunately, we need to be much more careful here. While gas for ARM
> > does support -mcpu, -march and -mfpu, this is not true for other ISAs.
> > Looking at the man page, it seems that the supported flags are:
> >
> > ARM: -mcpu=, -march=, -mfpu=
> > i386: -march= (-mtune=: should we map -mcpu= to -mtune=?)
> > MIPS: (same as i386)
> > PowerPC: supports cpu, but as -m<cpu> (meaning -m403|-m405|-mppc64|...)
> > SPARC: supports arch, but as -xarch=
> >
> > How do you think we should fix this?
>
> Well, we can make the set of forwarded options dependent on the target.
>

Yea, we need each target to manage the translation of these options.


> But what's the failing use case? I mean, why do you pass these options
> to the compiler in the first place?
>

-mcpu is a supported driver option? It should DTRT for every platform, and
that platform's assembler.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120423/1d01a10e/attachment.html>