This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Set correct probability for ORDER/UNORDER jumps
- From: Dehao Chen <dehao at google dot com>
- To: "H.J. Lu" <hjl dot tools at gmail dot com>
- Cc: Jakub Jelinek <jakub at redhat dot com>, Xinliang David Li <davidxl at google dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, Steven Bosscher <stevenb dot gcc at gmail dot com>
- Date: Fri, 21 Mar 2014 14:13:38 -0700
- Subject: Re: [PATCH] Set correct probability for ORDER/UNORDER jumps
- Authentication-results: sourceware.org; auth=none
- References: <CAO2gOZWhfyPKEmaEMBTap3e1RTAfsi6++J=BgY+C8gcmFfRT4w at mail dot gmail dot com> <CAAkRFZKioePmB+KHNwwfub0scFWie7gxotPQ6h_MXgmCQgix5g at mail dot gmail dot com> <CAO2gOZWLUUEAmBDWthEmsiMDnRi565sfb7SLn5rur4=GkD8gWA at mail dot gmail dot com> <20140124185706 dot GU892 at tucnak dot redhat dot com> <CAMe9rOqsWDTc48HR16puUnOS5GuK=pupO7PFxN1R=xbQWa+ozQ at mail dot gmail dot com> <CAO2gOZV8DiimCQ+dRT9a9bU3ZTzvcHp4A81kHpgQBJNpGjRkRg at mail dot gmail dot com> <CAO2gOZWLgWocJn9v_k2fv8h1D0kPZN4XuJxuir=iNoj1NWBnxQ at mail dot gmail dot com>
ping ^2...
Dehao
On Mon, Feb 10, 2014 at 8:35 AM, Dehao Chen <dehao@google.com> wrote:
> ping...
>
> Dehao
>
> On Fri, Jan 24, 2014 at 1:54 PM, Dehao Chen <dehao@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.tools@gmail.com> wrote:
>>> On Fri, Jan 24, 2014 at 10:57 AM, Jakub Jelinek <jakub@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.