[PATCH] Make the driver accept all four variants of the target option

Richard Barton richard.barton at arm.com
Thu Feb 19 03:47:31 PST 2015

I think that having -target=<triple> and --target <triple> only is awkward.
Why have two version of the same option with different rules on whether they
accept = or spaced arguments? 

I think Gabor's patch makes sense as it does not break any backwards
compatibility within clang itself and will only help people using the tool. 

The patch itself leaves some duplication in the Options.td file. Can't we
combine "target_legacy_spelling_EQ" with "target"? And perhaps rename them
both to just "target" and "target_EQ"?


> -----Original Message-----
> From: cfe-commits-bounces at cs.uiuc.edu [mailto:cfe-commits-
> bounces at cs.uiuc.edu] On Behalf Of Meador Inge
> Sent: 18 February 2015 18:09
> To: gaborb at inf.u-szeged.hu; Kristof Beyls; renato.golin at linaro.org
> Cc: cfe-commits at cs.uiuc.edu
> Subject: Re: [PATCH] Make the driver accept all four variants of the
> option
> Perhaps I am missing something, but how does having four variants for the
> option make it less confusing as a cross compiler?  Do we have a
> on which clang long options have '=' forms?
>   rL LLVM
> http://reviews.llvm.org/D7730
>   http://reviews.llvm.org/settings/panel/emailpreferences/
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

More information about the cfe-commits mailing list