[PATCH] Remove inefficient branchless conditional negate optimization
Jiong Wang
jiong.wang@arm.com
Fri Mar 6 16:37:00 GMT 2015
Wilco Dijkstra writes:
>> Jeff Law wrote:
>> Can you move pr45685.c into gcc.target/i386?
>>
>> I know Richi said next stage1, but given this fixes a performance
>> regression for ARM and it's reverting rather than adding new code, I
>> think this is OK for the trunk with the testcase moved.
>>
>> So, OK with the testcase moved into gcc.target/i386/
>
> I've moved it and changed the compile condition:
>
> /* { dg-do compile { target { ! { ia32 } } } } */
>
> Jiong, can you commit this please?
Done as 221246.
>
> Wilco
>
> 2015-03-06 Wilco Dijkstra <wdijkstr@arm.com>
>
> * gcc/tree-ssa-phiopt.c (neg_replacement): Remove.
> (tree_ssa_phiopt_worker): Remove negate optimization.
> * gcc/testsuite/gcc.dg/tree-ssa/pr45685.c: Move to gcc.target/i386.
> * gcc/testsuite/gcc.target/aarch64/csneg-1.c (test_csneg_cmp):
> New test.
> * gcc/gcc.target/i386/pr45685.c: Moved test, check for conditional
> move on x64.
--
Regards,
Jiong
More information about the Gcc-patches
mailing list