[Patch] Missed evaluation in "type" parameter of va_arg

[Richard Smith]

Please add a triple to your test (right now, it'd fail on targets where
'int' isn't i32). Otherwise, LGTM.


On Mon, Apr 7, 2014 at 11:26 AM, Rahul Jain <1989.rahuljain at gmail.com>wrote:

>
> Hi Dmitri,
>
> Thanks for replying.
>
> Updated patch with test case.
> Please help review the same.
>
> Thanks,
> Rahul
>
>
> On Fri, Apr 4, 2014 at 8:46 PM, Dmitri Gribenko <gribozavr at gmail.com>wrote:
>
>> On Wed, Apr 2, 2014 at 6:55 AM, Rahul Jain <1989.rahuljain at gmail.com>
>> wrote:
>> > Gentle ping!
>> >
>> > Please if someone could help review this small patch!
>>
>> Hello Rahul,
>>
>> This patch needs a testcase.
>>
>> 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 --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140413/224b59da/attachment.html>