[patch] Ignore -fstrength-reduce

[Chandler Carruth]

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>
> wrote:
> >> > I would not try to re-organize in a patch to add one flag. I suspect
> we
> >> > 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
> are
> > committed to doing a significant re-organization. We have a tendancy in
> this
> > 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...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20131021/5c51cb9f/attachment.html>