ping... Dehao
On Fri, Jan 24, 2014 at 1:54 PM, Dehao Chen <de...@google.com> wrote: > Thanks, test updated: > > Index: gcc/testsuite/gcc.dg/predict-8.c > =================================================================== > --- gcc/testsuite/gcc.dg/predict-8.c (revision 0) > +++ gcc/testsuite/gcc.dg/predict-8.c (revision 0) > @@ -0,0 +1,12 @@ > +/* { dg-do compile { target { i?86-*-* x86_64-*-* } } } */ > +/* { dg-options "-O2 -fdump-rtl-expand" } */ > + > +int foo(float a, float b) { > + if (a == b) > + return 1; > + else > + return 2; > +} > + > +/* { dg-final { scan-rtl-dump-times "REG_BR_PROB 100" 1 "expand"} } */ > +/* { dg-final { cleanup-rtl-dump "expand" } } */ > > On Fri, Jan 24, 2014 at 11:38 AM, H.J. Lu <hjl.to...@gmail.com> wrote: >> On Fri, Jan 24, 2014 at 10:57 AM, Jakub Jelinek <ja...@redhat.com> wrote: >>> On Fri, Jan 24, 2014 at 10:20:53AM -0800, Dehao Chen wrote: >>>> --- gcc/testsuite/gcc.dg/predict-8.c (revision 0) >>>> +++ gcc/testsuite/gcc.dg/predict-8.c (revision 0) >>>> @@ -0,0 +1,12 @@ >>>> +/* { dg-do compile { target { x86_64-*-* } } } */ >>> >>> If you want it for x86_64 64-bit, then >>> /* { dg-do compile { target { { i?86-*-* x86_64-*-* } && lp64 } } } */ >>> >> >> It should be "! { ia32 }" instead of "lp64" unless it doesn't work >> for x32. >> >> -- >> H.J.