[patch] Ignore -fstrength-reduce

Rafael EspĂ­ndola rafael.espindola at gmail.com
Mon Oct 21 11:35:41 PDT 2013


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 --------------
A non-text attachment was scrubbed...
Name: t.patch
Type: application/octet-stream
Size: 1449 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20131021/7368c885/attachment.obj>


More information about the cfe-commits mailing list