Hi Richard, Thanks for your valuable input. Updated the patch accordingly.
Please if you could commit the same for me. Thanks, Rahul On Mon, Apr 14, 2014 at 5:08 AM, Richard Smith <[email protected]>wrote: > 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 <[email protected]>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 <[email protected]>wrote: >> >>> On Wed, Apr 2, 2014 at 6:55 AM, Rahul Jain <[email protected]> >>> 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 <[email protected]>*/ >>> >> >> >
vla_3.patch
Description: Binary data
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
