[PATCH] add the -mrecip driver flag and process its options

hfinkel at anl.gov hfinkel at anl.gov
Sat May 16 13:09:22 PDT 2015


In http://reviews.llvm.org/D8989#173899, @hfinkel wrote:

> In http://reviews.llvm.org/D8989#173894, @spatel wrote:
>
> > In http://reviews.llvm.org/D8989#173790, @hfinkel wrote:
> >
> > > GCC, to combine the x86 and PPC options, accepts the keys all, default, div (for both divf and divd), divf, divd, rsqrt (for both rsqrtf and rsqrtd), rsqrtf, rsqrtd, and then these same things with the 'vec-' prefix. '!' is used as the prefix to indicate disabling (not '-' as you have here).
> > >
> > > I'm fine with the :<n> syntax extension, but I'd rather use GCC's syntax here as a base for compatibility.
> >
> >
> > Ok - I missed the f/d shortcut. I wasn't sure if compatibility with GCC (the '!') trumped the existing enable/disable (+/-) syntax that we use for target features. Let me know if we should keep the +/- for the llc command-line or switch that to ! as well.
>
>
> I'd rather you keep the llc syntax the same.


I realize that's ambiguous. I'd rather the llc syntax match the Clang syntax.


http://reviews.llvm.org/D8989

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list