On Tue, Aug 27, 2013 at 4:41 PM, Rafael Espíndola <
[email protected]> wrote:

> > Any reason not to remove this, and instead look at the argument to the
> OPT_O
> > option?
> >
> > (Likewise, why do we have a separate -O0 flag?)
>
> Just convenience. It is easier to write
>
> A->getOption().matches(options::OPT_O4)
>
> Than to check it matches OPT_O, get the value, check if it is integer and
> == 4.
>


Maybe we should warn for all integers >= 4 though.

-- Sean Silva


>
> Cheers,
> Rafael
> _______________________________________________
> cfe-commits mailing list
> [email protected]
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to