[patch] Ignore -fstrength-reduce
chandlerc at google.com
Mon Oct 21 11:45:56 PDT 2013
Generally looks fine...
+// RUN: -fstrength-reduce -fstrength-reduce
On Mon, Oct 21, 2013 at 11:35 AM, Rafael Espíndola <
rafael.espindola at gmail.com> wrote:
> OK, new patch attached.
> On 21 October 2013 14:33, Chandler Carruth <chandlerc at google.com> wrote:
> > On Mon, Oct 21, 2013 at 11:30 AM, Rafael Espíndola
> > <rafael.espindola at gmail.com> wrote:
> >> On 21 October 2013 14:27, Chandler Carruth <chandlerc at google.com>
> >> > I would not try to re-organize in a patch to add one flag. I suspect
> >> > want
> >> > to ignore many if not most of the flags we currently ignore.
> >> Probably. This is the one I am sure about it. The idea is that we move
> >> or implement them as we go.
> > I would *much* rather not add any more sections to this file unless you
> > committed to doing a significant re-organization. We have a tendancy in
> > file to start a new organizational scheme but not make it far enough for
> > others to realize it when editting this file.
> > So, I would add it to the poorly-sorted list as yet-another ignored flag,
> > and do a separate patch that attempts to group and categorize things
> > separately.
> >> Cheers,
> >> Rafael
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the cfe-commits