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
