[cfe-commits] [PATCH] Implement -Wvla correctly

Dmitri Gribenko gribozavr at gmail.com
Wed Jan 23 11:53:47 PST 2013


On Tue, Jan 15, 2013 at 7:31 PM, Dmitri Gribenko <gribozavr at gmail.com> wrote:
> On Tue, Jan 8, 2013 at 8:38 PM, Rafael EspĂ­ndola
> <rafael.espindola at gmail.com> wrote:
>> On 8 January 2013 11:31, Dmitri Gribenko <gribozavr at gmail.com> 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?

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 <gribozavr at gmail.com>*/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: implement-wvla-correctly-v3.patch
Type: application/octet-stream
Size: 4513 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130123/a37d6573/attachment.obj>


More information about the cfe-commits mailing list