Slightly tweaked and committed as r206223. Thanks!
On Mon, Apr 14, 2014 at 7:55 AM, Rahul Jain <[email protected]>wrote: > > 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]>*/ >>>> >>> >>> >> >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
