[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