[Bug target/18154] Inefficient max/min code for PowerPC

2017-11-21 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=18154 --- Comment #13 from Segher Boessenkool --- Trunk now generates isel for power9.

[Bug target/18154] Inefficient max/min code for PowerPC

2016-08-23 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=18154 --- Comment #12 from Segher Boessenkool --- (Never mind those last "addc" insn, they can just as well be plain "add", I pasted the wrong ones).

[Bug target/18154] Inefficient max/min code for PowerPC

2016-08-23 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=18154 --- Comment #11 from Segher Boessenkool --- The signed version can be done in four insns: 1: subfc r5,r3,r4 subfe r6,r6,r6 and r7,r6,r5 addcr8,r7,r3 (superopt finds 16 versions, all similar). The unsign

[Bug target/18154] Inefficient max/min code for PowerPC

2016-01-30 Thread dje at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=18154 --- Comment #10 from David Edelsohn --- isel is not generally performance win for Power using GCC. It is enabled for LLVM because LLVM has a simplistic basic block scheduler and isel allows LLVM to form larger basic blocks to provide the schedul

[Bug target/18154] Inefficient max/min code for PowerPC

2016-01-30 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=18154 --- Comment #9 from Martin Sebor --- I noticed while looking at an unrelated bug that when targeting power7 or power8 Clang makes use of the isel instruction and emits the following: min:# @min cmpw

[Bug target/18154] Inefficient max/min code for PowerPC

2016-01-27 Thread dje at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=18154 David Edelsohn changed: What|Removed |Added Status|WAITING |NEW CC|

[Bug target/18154] Inefficient max/min code for PowerPC

2016-01-27 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=18154 Martin Sebor changed: What|Removed |Added Target|powerpc-*-* |powerpc*-*-* Status|NEW

[Bug target/18154] Inefficient max/min code for PowerPC

2005-04-24 Thread pinskia at gcc dot gnu dot org
--- Additional Comments From pinskia at gcc dot gnu dot org 2005-04-24 14:31 --- On the mainline, we now produce: cmpw cr7,r3,r4 blelr- cr7 mr r3,r4 blr -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=18154

[Bug target/18154] Inefficient max/min code for PowerPC

2004-10-27 Thread pinskia at gcc dot gnu dot org
--- Additional Comments From pinskia at gcc dot gnu dot org 2004-10-27 23:52 --- I should note when I was doing SPEC work, using subfc/subfe did not help SPEC at all (I tried to change the source and also rs6000.md). -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=18154

[Bug target/18154] Inefficient max/min code for PowerPC

2004-10-27 Thread geoffk at gcc dot gnu dot org
--- Additional Comments From geoffk at gcc dot gnu dot org 2004-10-27 23:45 --- I'm not sure that subfc/subfe is going to be cheaper than a compare and a branch, even if the branch is mispredicted half the time. Do you have timing results? -- http://gcc.gnu.org/bugzilla/show_bug.c

[Bug target/18154] Inefficient max/min code for PowerPC

2004-10-26 Thread dje at gcc dot gnu dot org
--- Additional Comments From dje at gcc dot gnu dot org 2004-10-26 21:25 --- Also, do not enable when optimizing for size. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=18154

[Bug target/18154] Inefficient max/min code for PowerPC

2004-10-26 Thread dje at gcc dot gnu dot org
--- Additional Comments From dje at gcc dot gnu dot org 2004-10-26 20:06 --- XLC chooses the straight-line code sequence versus compare and branch based on a cost model. This should not be a uniform change in behavior for PowerPC. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=18154

[Bug target/18154] Inefficient max/min code for PowerPC

2004-10-25 Thread pinskia at gcc dot gnu dot org
--- Additional Comments From pinskia at gcc dot gnu dot org 2004-10-26 04:25 --- Confirmed. -- What|Removed |Added Status|UNCONFIRMED |NEW E