On Wed, Jan 23, 2013 at 10:00 PM, Chandler Carruth <[email protected]> wrote:
> On Wed, Jan 23, 2013 at 11:53 AM, Dmitri Gribenko <[email protected]>
> wrote:
>>
>> On Tue, Jan 15, 2013 at 7:31 PM, Dmitri Gribenko <[email protected]>
>> wrote:
>> > On Tue, Jan 8, 2013 at 8:38 PM, Rafael Espíndola
>> > <[email protected]> wrote:
>> >> On 8 January 2013 11:31, Dmitri Gribenko <[email protected]> wrote:
>> >>> Hello Rafael,
>> >>>
>> >>> I have added the requested tests.
>> >>
>> >> Cool. The patch look good to me. Just give chandler a day or so to
>> >> comment on it. The google style bans VLAs, so he might have an opinion
>> >> on the patch.
>> >
>> > Ping.
>>
>> Ping^2.
>>
>> Chandler: do you have concerns about this?
>
> Sorry, I missed Rafael's comment. Google's style should have no bearing on a
> warning. That doesn't make any sense.
>
> This patch makes a lot of sense. I don't know what could be bad about it.
>
> I assume the idea of warning about vla in general is for GCC compatibility
> or security concerns? Seems reasonabl eto keep that functionality.
Right, GCC implemented this to warn on all VLAs. We can argue if this
warning belongs to the compiler or the mythical 'style checker', but
we can trivially implement this to be compatible with GCC.
> Anyways, don't wait for me, commit!
Thanks for the review, committed r173286.
Dmitri
--
main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
(j){printf("%d\n",i);}}} /*Dmitri Gribenko <[email protected]>*/
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
