[Bug rtl-optimization/78638] [7 regression] test cases gcc.target/powerpc/rlwimi-0.c and rlwimi-2.c fail starting with r243000

2016-12-02 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78638 Segher Boessenkool changed: What|Removed |Added CC||krebbel at gcc dot gnu.org --- Comm

[Bug rtl-optimization/78561] Constant pool size (offset) can become stale where constant pool entires become unused

2016-12-03 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78561 Segher Boessenkool changed: What|Removed |Added CC||segher at gcc dot gnu.org

[Bug rtl-optimization/78561] Constant pool size (offset) can become stale where constant pool entires become unused

2016-12-03 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78561 --- Comment #5 from Segher Boessenkool --- Oh btw, you forgot to commit the testcase in 2/2.

[Bug rtl-optimization/78342] [7 Regression] ICE in rtl_verify_bb_insns, at cfgrtl.c:2657 (error: flow control insn inside a basic block)

2016-12-04 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78342 --- Comment #11 from Segher Boessenkool --- No, we probably still want backports.

[Bug rtl-optimization/78610] [7 Regression] ICE in rtl_verify_bb_insns, at cfgrtl.c:2656 (error: flow control insn inside a basic block)

2016-12-04 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78610 --- Comment #4 from Segher Boessenkool --- Nope, backports are needed.

[Bug rtl-optimization/78561] Constant pool size (offset) can become stale where constant pool entires become unused

2016-12-05 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78561 --- Comment #8 from Segher Boessenkool --- I usually use --disable-libgomp, but otherwise everything default (well, --enable-languages=all,ada,go,obj-c++).

[Bug rtl-optimization/78561] Constant pool size (offset) can become stale where constant pool entires become unused

2016-12-05 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78561 --- Comment #10 from Segher Boessenkool --- gcc112 is LE, it cannot run 32-bit at all :-) Try gcc110, that works.

[Bug rtl-optimization/78561] Constant pool size (offset) can become stale where constant pool entires become unused

2016-12-05 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78561 --- Comment #12 from Segher Boessenkool --- It still happens here, also on gcc110. Note you need --disable-werror, to avoid another bootstrap error. Did you perchance use --disable-bootstrap?

[Bug rtl-optimization/78561] Constant pool size (offset) can become stale where constant pool entires become unused

2016-12-05 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78561 --- Comment #14 from Segher Boessenkool --- I used trunk. --disable-bootstrap fails the same, just much faster ;-) Maybe the binutils etc. version matters?

[Bug rtl-optimization/78561] Constant pool size (offset) can become stale where constant pool entires become unused

2016-12-06 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78561 --- Comment #17 from Segher Boessenkool --- (In reply to James Greenhalgh from comment #16) > Created attachment 40267 [details] > Proposed Patch > > Would you mind testing the attached to see if it fixes your issue? Bootstrapped fine, regressi

[Bug rtl-optimization/78626] [7 Regression] ICE in rtl_verify_bb_insns, at cfgrtl.c:2656 (error: flow control insn inside a basic block)

2016-12-06 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78626 --- Comment #2 from Segher Boessenkool --- I have tested something similar, and it does work, but it prevents any optimisation by cprop of any trap_if, also if it would not turn into an unconditional trap. This is pretty bad :-(

[Bug rtl-optimization/78626] [7 Regression] ICE in rtl_verify_bb_insns, at cfgrtl.c:2656 (error: flow control insn inside a basic block)

2016-12-06 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78626 --- Comment #5 from Segher Boessenkool --- (In reply to Bernd Schmidt from comment #3) > Not sure it's that bad really. An unconditional trap is pretty much by > definition not performance-critical. Sure, but this was prohibiting propagating any

[Bug target/72717] [5/6/7 Regression] ICE: in emit_move_insn, at expr.c:3693 with vector shift @ powerpc64le

2016-12-07 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72717 --- Comment #4 from Segher Boessenkool --- That works I guess... Please test on BE and 32-bit as well. Oh, and no parens in a = (b) ? c : d; (for simple b at least).

[Bug rtl-optimization/78626] [7 Regression] ICE in rtl_verify_bb_insns, at cfgrtl.c:2656 (error: flow control insn inside a basic block)

2016-12-07 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78626 --- Comment #7 from Segher Boessenkool --- (In reply to Bernd Schmidt from comment #4) > Created attachment 40269 [details] > Candidate patch That looks great :-)

[Bug rtl-optimization/78626] [7 Regression] ICE in rtl_verify_bb_insns, at cfgrtl.c:2656 (error: flow control insn inside a basic block)

2016-12-07 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78626 --- Comment #8 from Segher Boessenkool --- ... and works fine, too! ms: stwu 1,-32(1) lis 9,qs@ha lwz 9,qs@l(9) twnei 9,0 .L6: b .L6 Nice :-)

[Bug target/77957] [5/6 Regression] Undefined .LCTOC0 with -fstack-protector-strong -mminimal-toc -O0 on ppc64

2016-12-07 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77957 --- Comment #7 from Segher Boessenkool --- Author: segher Date: Wed Dec 7 23:07:58 2016 New Revision: 243415 URL: https://gcc.gnu.org/viewcvs?rev=243415&root=gcc&view=rev Log: rs6000: Don't forget to initialize the TOC (PR77957) The code gener

[Bug target/77957] [5/6 Regression] Undefined .LCTOC0 with -fstack-protector-strong -mminimal-toc -O0 on ppc64

2016-12-07 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77957 --- Comment #8 from Segher Boessenkool --- Author: segher Date: Wed Dec 7 23:11:23 2016 New Revision: 243416 URL: https://gcc.gnu.org/viewcvs?rev=243416&root=gcc&view=rev Log: rs6000: Don't forget to initialize the TOC (PR77957) The code gener

[Bug target/77957] [5/6 Regression] Undefined .LCTOC0 with -fstack-protector-strong -mminimal-toc -O0 on ppc64

2016-12-07 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77957 --- Comment #9 from Segher Boessenkool --- Fixed on all open branches.

[Bug target/77957] [5/6 Regression] Undefined .LCTOC0 with -fstack-protector-strong -mminimal-toc -O0 on ppc64

2016-12-07 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77957 Segher Boessenkool changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug rtl-optimization/78638] [7 regression] test cases gcc.target/powerpc/rlwimi-0.c and rlwimi-2.c fail starting with r243000

2016-12-07 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78638 --- Comment #6 from Segher Boessenkool --- Author: segher Date: Thu Dec 8 00:09:01 2016 New Revision: 243420 URL: https://gcc.gnu.org/viewcvs?rev=243420&root=gcc&view=rev Log: simplify-rtx: Fix the last fix (PR78638) I managed to get the last

[Bug rtl-optimization/78638] [7 regression] test cases gcc.target/powerpc/rlwimi-0.c and rlwimi-2.c fail starting with r243000

2016-12-07 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78638 Segher Boessenkool changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug rtl-optimization/78610] [7 Regression] ICE in rtl_verify_bb_insns, at cfgrtl.c:2656 (error: flow control insn inside a basic block)

2016-12-09 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78610 Segher Boessenkool changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug rtl-optimization/78727] [7 Regression] ICE in rtl_verify_bb_insns, at cfgrtl.c:2656 (error: flow control insn inside a basic block)

2016-12-09 Thread segher at gcc dot gnu.org
||2016-12-09 CC||segher at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #4 from Segher Boessenkool --- Confirmed.

[Bug target/78683] [ppc] __builtin_ctzl code gen can be improved for -mcpu=power8 and earlier

2016-12-09 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78683 --- Comment #1 from Segher Boessenkool --- Author: segher Date: Fri Dec 9 19:31:06 2016 New Revision: 243499 URL: https://gcc.gnu.org/viewcvs?rev=243499&root=gcc&view=rev Log: rs6000: clz/ctz/ffs improvement (PR78683) On CPUs that implement po

[Bug middle-end/17308] nonnull attribute not as useful as it could be

2016-12-14 Thread segher at gcc dot gnu.org
||segher at gcc dot gnu.org Resolution|FIXED |--- --- Comment #18 from Segher Boessenkool --- This gives a warning in powerpc-linuc (which breaks bootstrap), when compiling tree-inline.c: /home/segher/src/gcc/gcc/vec.h:1613:5: error: argument 1

[Bug middle-end/17308] nonnull attribute not as useful as it could be

2016-12-14 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=17308 --- Comment #19 from Segher Boessenkool --- powerpc64-linux, even.

[Bug bootstrap/78817] stage2 bootstrap failure in vec.h:1613:5: error: argument 1 null where non-null expected after r243661

2016-12-14 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78817 Segher Boessenkool changed: What|Removed |Added CC||segher at gcc dot gnu.org

[Bug bootstrap/78817] stage2 bootstrap failure in vec.h:1613:5: error: argument 1 null where non-null expected after r243661

2016-12-15 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78817 --- Comment #3 from Segher Boessenkool --- It is a warning (as well as a bootstrap comparison error) on powerpc64-linux. You tested on powerpc64le-linux, different animal.

[Bug target/78764] CPU type option discarded on 32-bit powerpc

2016-12-15 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78764 Segher Boessenkool changed: What|Removed |Added CC||segher at gcc dot gnu.org

[Bug target/71216] [4.9/5/6/7 Regression] Incorrect PPC assembly due to inserted .machine pseudo-op

2016-12-15 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71216 Segher Boessenkool changed: What|Removed |Added CC||rin at NetBSD dot org --- Comment #

[Bug target/78764] CPU type option discarded on 32-bit powerpc

2016-12-15 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78764 Segher Boessenkool changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug target/71216] [4.9/5/6/7 Regression] Incorrect PPC assembly due to inserted .machine pseudo-op

2016-12-16 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71216 --- Comment #8 from Segher Boessenkool --- Hi Rin, > However, I have a question on this fix. How about the case where > "-Wa,-mXXX" option is given without "-mcpu=YYY" option specified? That might or might not work; the user had better know wha

[Bug rtl-optimization/78751] [7 Regression] ICE in extract_insn, at recog.c:2311 (error: unrecognizable insn) w/ -Os -misel

2016-12-19 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78751 --- Comment #2 from Segher Boessenkool --- In this testcase ifcvt happens upon a branch like: (jump_insn 28 27 65 2 (set (pc) (if_then_else (eq (reg:CCEQ 183) (const_int 0 [0])) (label_ref:SI 65)

[Bug target/78683] [ppc] __builtin_ctzl code gen can be improved for -mcpu=power8 and earlier

2016-12-19 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78683 Segher Boessenkool changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug rtl-optimization/78751] [7 Regression] ICE in extract_insn, at recog.c:2311 (error: unrecognizable insn) w/ -Os -misel

2016-12-20 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78751 --- Comment #3 from Segher Boessenkool --- Created attachment 40383 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=40383&action=edit patch I use this patch as workaround. It, well, sucks. But it does work.

[Bug target/77345] [7 Regression] Segmentation fault w/ -misel -O1 (and above)

2016-12-21 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77345 --- Comment #5 from Segher Boessenkool --- It looks very much like PR71724 indeed, but I cannot get this one to fail.

[Bug rtl-optimization/71724] [5/6/7 Regression] ICE: Segmentation fault, deep recursion between combine_simplify_rtx and subst

2016-12-21 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71724 --- Comment #5 from Segher Boessenkool --- I am using the following, which also fixes the infinite loop, and seems to not regress code quality much at all (I found *one* pattern where it made things one machine insn worse, involving a define_insn

[Bug middle-end/78904] zero-extracts are not effective

2016-12-22 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78904 Segher Boessenkool changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug target/71216] [4.9/5/6/7 Regression] Incorrect PPC assembly due to inserted .machine pseudo-op

2016-12-22 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71216 --- Comment #10 from Segher Boessenkool --- (In reply to Rin Okuyama from comment #9) > > > However, I have a question on this fix. How about the case where > > > "-Wa,-mXXX" option is given without "-mcpu=YYY" option specified? > > > > That mig

[Bug middle-end/78904] zero-extracts are not effective

2016-12-22 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78904 --- Comment #3 from Segher Boessenkool --- (In reply to Uroš Bizjak from comment #2) > No, unfortunately the above is not a valid x86 insn. x86 has two-operand > instructions, so output has to match one of the operands. But these are pseudos. >

[Bug middle-end/78904] zero-extracts are not effective

2016-12-22 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78904 --- Comment #7 from Segher Boessenkool --- Ah, "high byte" registers are never a separate register in the i386 backend, I see. combine would need to combine four insns to arrive at your current pattern, but it doesn't try because it thinks they

[Bug rtl-optimization/78342] ICE in rtl_verify_bb_insns, at cfgrtl.c:2657 (error: flow control insn inside a basic block)

2017-01-05 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78342 Segher Boessenkool changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug rtl-optimization/78626] [7 Regression] ICE in rtl_verify_bb_insns, at cfgrtl.c:2656 (error: flow control insn inside a basic block)

2017-01-09 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78626 --- Comment #11 from Segher Boessenkool --- It is ready to be committed AFAIK; same for the related PR78727.

[Bug target/68803] gcc.vect/powerpc/20050603-3.c failures since r230167

2017-01-09 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68803 Segher Boessenkool changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug target/79039] New: builtins-3-p9.c fails with -m32

2017-01-09 Thread segher at gcc dot gnu.org
Assignee: unassigned at gcc dot gnu.org Reporter: segher at gcc dot gnu.org Target Milestone: --- The testcase fails with -m32, as reported by Andreas in https://gcc.gnu.org/ml/gcc-patches/2017-01/msg00017.html . This is because long is just 32 bits with -m32, so generates the same code

[Bug target/79039] builtins-3-p9.c fails with -m32

2017-01-09 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79039 Segher Boessenkool changed: What|Removed |Added Target||powerpc*-*-* Status|UNC

[Bug target/79040] New: vec_cntlz redefined

2017-01-09 Thread segher at gcc dot gnu.org
: unassigned at gcc dot gnu.org Reporter: segher at gcc dot gnu.org Target Milestone: --- /home/segher/build/tot-master/gcc/include/altivec.h:392:0: warning: "vec_cntlz" redefined #define vec_cntlz __builtin_vec_vctz /home/segher/build/tot-master/gcc/include/altivec.h:352:0: not

[Bug target/79040] vec_cntlz redefined

2017-01-09 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79040 Segher Boessenkool changed: What|Removed |Added Target||powerpc*-*-* Status|UNC

[Bug target/61729] FAIL: g++.dg/abi/scoped1.C -std=gnu++11 execution test

2017-01-10 Thread segher at gcc dot gnu.org
||2017-01-10 CC||segher at gcc dot gnu.org Ever confirmed|0 |1

[Bug target/71847] powerpc64le: Potential rlwinm optimisation

2017-01-10 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71847 Segher Boessenkool changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug target/79058] [7 Regression] ARM: internal compiler error: in extract_constrain_insn, at recog.c:2213

2017-01-12 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79058 --- Comment #9 from Segher Boessenkool --- With the code and flags in comment 2 i get a segmentation fault, instead (with a powerpc64-linux host), somewhere during LRA. insn 10 is === (insn 10 8 11 2 (set (reg:DI 120) (and:DI (subreg:DI

[Bug rtl-optimization/78751] [7 Regression] ICE in extract_insn, at recog.c:2311 (error: unrecognizable insn) w/ -Os -misel

2017-01-12 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78751 --- Comment #5 from Segher Boessenkool --- Oh, the patch isn't ugly, just the resulting code is :-)

[Bug target/79066] [7 Regression] non-PIC code generated for powerpc glibc with -fpic

2017-01-12 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79066 --- Comment #3 from Segher Boessenkool --- Confirmed. The constant is forced to mem in LRA.

[Bug target/79066] [7 Regression] non-PIC code generated for powerpc glibc with -fpic

2017-01-12 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79066 --- Comment #5 from Segher Boessenkool --- -mno-lra calls rs6000_emit_move to load the address of the const mem it creates; -mlra does not. It should, but how what where.

[Bug rtl-optimization/71724] [5/6/7 Regression] ICE: Segmentation fault, deep recursion between combine_simplify_rtx and subst

2017-01-12 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71724 --- Comment #6 from Segher Boessenkool --- *** Bug 77345 has been marked as a duplicate of this bug. ***

[Bug target/77345] [7 Regression] Segmentation fault w/ -misel -O1 (and above)

2017-01-12 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77345 Segher Boessenkool changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug target/72749] [7 Regression] ICE: verify_flow_info failed (error: wrong amount of branch edges after conditional jump in bb 5) w/ -O2 -fsched2-use-superblocks

2017-01-12 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72749 --- Comment #9 from Segher Boessenkool --- (In reply to Arseny Solokha from comment #7) > I wonder if the following ICE is somehow related to the one reported here. > I'll file a new PR if it's not. This is a different bug (it still happens with

[Bug target/72749] [7 Regression] ICE: verify_flow_info failed (error: wrong amount of branch edges after conditional jump in bb 5) w/ -O2 -fsched2-use-superblocks

2017-01-14 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72749 --- Comment #13 from Segher Boessenkool --- I have a patch bootstrapping, let's not close this yet.

[Bug target/61729] FAIL: g++.dg/abi/scoped1.C -std=gnu++11 execution test

2017-01-14 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61729 --- Comment #1 from Segher Boessenkool --- This testcase uses a 2-byte scoped enum, which doesn't get the integer promotions if I read the C++ standard correctly -- but it is passed via varargs, and the target code expects that to be promoted alw

[Bug rtl-optimization/78751] [7 Regression] ICE in extract_insn, at recog.c:2311 (error: unrecognizable insn) w/ -Os -misel

2017-01-15 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78751 --- Comment #6 from Segher Boessenkool --- Author: segher Date: Sun Jan 15 17:03:55 2017 New Revision: 244476 URL: https://gcc.gnu.org/viewcvs?rev=244476&root=gcc&view=rev Log: ifcvt: Don't make invalid insns for a cond trap (PR78751) As shown

[Bug target/72749] [7 Regression] ICE: verify_flow_info failed (error: wrong amount of branch edges after conditional jump in bb 5) w/ -O2 -fsched2-use-superblocks

2017-01-15 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72749 --- Comment #14 from Segher Boessenkool --- Author: segher Date: Sun Jan 15 17:06:00 2017 New Revision: 244477 URL: https://gcc.gnu.org/viewcvs?rev=244477&root=gcc&view=rev Log: Make rtl_split_edge work for jumps that fall through (PR72749) If

[Bug target/72749] [7 Regression] ICE: verify_flow_info failed (error: wrong amount of branch edges after conditional jump in bb 5) w/ -O2 -fsched2-use-superblocks

2017-01-15 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72749 Segher Boessenkool changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug rtl-optimization/78751] [7 Regression] ICE in extract_insn, at recog.c:2311 (error: unrecognizable insn) w/ -Os -misel

2017-01-15 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78751 Segher Boessenkool changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug target/61729] FAIL: g++.dg/abi/scoped1.C -std=gnu++11 execution test

2017-01-16 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61729 --- Comment #3 from Segher Boessenkool --- Okay, I'll make it work for SVR4 in the rs6000 backend then. The generic code makes suboptimal code, many ABIs need to update (even those that haven't changed for 25 years), and more backends will need

[Bug rtl-optimization/78952] Combine does not convert 8-bit sign-extract to a zero-extract for QImode operations

2017-01-17 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78952 --- Comment #3 from Segher Boessenkool --- Note avr has an "any_extract" code_iterator for this. That of course is a crutch but could be a workaround for you for now.

[Bug target/78875] -fstack-protector on powerpc64 now always use TLS, won't work for kernel/firmware

2017-01-17 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78875 --- Comment #2 from Segher Boessenkool --- Author: segher Date: Tue Jan 17 22:02:42 2017 New Revision: 244556 URL: https://gcc.gnu.org/viewcvs?rev=244556&root=gcc&view=rev Log: -mstack-protector-guard and friends (PR78875) Currently, on PowerPC

[Bug target/78875] -fstack-protector on powerpc64 now always use TLS, won't work for kernel/firmware

2017-01-18 Thread segher at gcc dot gnu.org
||7.0 Assignee|unassigned at gcc dot gnu.org |segher at gcc dot gnu.org Known to fail|7.0 | --- Comment #4 from Segher Boessenkool --- Should work on trunk now. Needs backports to all open release branches.

[Bug target/61729] FAIL: g++.dg/abi/scoped1.C -std=gnu++11 execution test

2017-01-18 Thread segher at gcc dot gnu.org
|unassigned at gcc dot gnu.org |segher at gcc dot gnu.org

[Bug target/79140] gcc.target/powerpc/ssp-1.c fails starting with its introduction in r244562

2017-01-18 Thread segher at gcc dot gnu.org
||2017-01-19 Assignee|unassigned at gcc dot gnu.org |segher at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #1 from Segher Boessenkool --- Confirmed. I have a patch.

[Bug target/77346] [7 Regression] ICE in push_reload, at reload.c:1350

2017-01-19 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77346 --- Comment #13 from Segher Boessenkool --- (In reply to Jeffrey A. Law from comment #12) > It does raise the question of how long we're going to support -mno-lra on > PPC. The plan is to remove it in GCC 8.

[Bug target/78875] -fstack-protector on powerpc64 now always use TLS, won't work for kernel/firmware

2017-01-19 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78875 --- Comment #5 from Segher Boessenkool --- Author: segher Date: Fri Jan 20 01:22:27 2017 New Revision: 244677 URL: https://gcc.gnu.org/viewcvs?rev=244677&root=gcc&view=rev Log: rs6000: Fix the new SSP guard configuration code (PR79140) I foolis

[Bug target/79140] gcc.target/powerpc/ssp-1.c fails starting with its introduction in r244562

2017-01-19 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79140 --- Comment #2 from Segher Boessenkool --- Author: segher Date: Fri Jan 20 01:22:27 2017 New Revision: 244677 URL: https://gcc.gnu.org/viewcvs?rev=244677&root=gcc&view=rev Log: rs6000: Fix the new SSP guard configuration code (PR79140) I foolis

[Bug target/61729] FAIL: g++.dg/abi/scoped1.C -std=gnu++11 execution test

2017-01-20 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61729 --- Comment #4 from Segher Boessenkool --- Author: segher Date: Sat Jan 21 03:11:49 2017 New Revision: 244740 URL: https://gcc.gnu.org/viewcvs?rev=244740&root=gcc&view=rev Log: rs6000: Small varargs for BE SVR4 (PR61729, PR77850) The varargs co

[Bug target/77850] FAIL: gcc.dg/compat/scalar-by-value-4 c_compat_x_tst.o-c_compat_y_tst.o execute

2017-01-20 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77850 --- Comment #1 from Segher Boessenkool --- Author: segher Date: Sat Jan 21 03:11:49 2017 New Revision: 244740 URL: https://gcc.gnu.org/viewcvs?rev=244740&root=gcc&view=rev Log: rs6000: Small varargs for BE SVR4 (PR61729, PR77850) The varargs co

[Bug target/79197] [5/6/7 Regression] ICE in extract_insn in gcc/recog.c:2311

2017-01-23 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79197 --- Comment #3 from Segher Boessenkool --- Ah I can reproduce it now... -mcpu=power7 -mno-popcntd

[Bug rtl-optimization/78559] [7 Regression] wrong code due to tree if-conversion?

2017-01-24 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78559 --- Comment #10 from Segher Boessenkool --- I am leaning toward accepting Bin's patch, but testing whether that hurts generated code too much still hasn't finished.

[Bug target/79211] ICE in extract_insn, at recog.c:2311

2017-01-26 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79211 --- Comment #1 from Segher Boessenkool --- I cannot reproduce this problem; are there any special (configure) flags I need?

[Bug target/79211] ICE in extract_insn, at recog.c:2311

2017-01-27 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79211 Segher Boessenkool changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug target/79211] ICE in extract_insn, at recog.c:2311

2017-01-27 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79211 --- Comment #4 from Segher Boessenkool --- Ah. That reg:SF 3 is not an fpr_reg_operand (3 is GPR 3). Now how did that happen...

[Bug target/79211] ICE in extract_insn, at recog.c:2311

2017-01-27 Thread segher at gcc dot gnu.org
|unassigned at gcc dot gnu.org |segher at gcc dot gnu.org

[Bug target/67288] [5/6/7 regression] non optimal simple function (useless additional shift/remove/shift/add)

2017-01-30 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67288 --- Comment #6 from Segher Boessenkool --- (In reply to Bernd Schmidt from comment #5) > (In reply to Christophe Leroy from comment #0) > > > The following section is just useless: (shift left 4 bits, remove 16, shift > > right 4 bits, add 1) >

[Bug middle-end/68664] [6/7 Regression] Speculative sqrt in c-ray main loop causes large slow down

2017-01-30 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68664 --- Comment #12 from Segher Boessenkool --- new_ready just adds insns to the ready list. High latency isn't directly a problem: if we can schedule a high latency insn early speculatively, that is a _good_ thing!

[Bug middle-end/68664] [6/7 Regression] Speculative sqrt in c-ray main loop causes large slow down

2017-01-31 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68664 --- Comment #14 from Segher Boessenkool --- I'm not sure how to read your remark. An insn where the result is not used is not on the critical path by definition; and you seem to be arguing for -fno-sched-spec by default?

[Bug target/79295] [7 regression] gcc.target/powerpc/bcd-3.c fails starting with r244942

2017-01-31 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79295 --- Comment #1 from Segher Boessenkool --- So operands[0] is wrong (should be an AltiVec reg, is an FP reg).

[Bug rtl-optimization/79279] combine/simplify_set: wrong call to REG_CANNOT_CHANGE_MODE_P

2017-02-02 Thread segher at gcc dot gnu.org
||2017-02-03 Resolution|WONTFIX |--- Assignee|unassigned at gcc dot gnu.org |segher at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #3 from Segher Boessenkool --- See https://gcc.gnu.org/ml/gcc

[Bug target/79354] [7 Regression] -mcpu=power8 -O2 generates power9 instruction on powerpc64le-linux

2017-02-02 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79354 --- Comment #4 from Segher Boessenkool --- That looks good to me. Mike, to you too?

[Bug rtl-optimization/68664] [6/7 Regression] Speculative sqrt in c-ray main loop causes large slow down

2017-02-03 Thread segher at gcc dot gnu.org
-end |rtl-optimization Assignee|unassigned at gcc dot gnu.org |segher at gcc dot gnu.org

[Bug rtl-optimization/68664] [6/7 Regression] Speculative sqrt in c-ray main loop causes large slow down

2017-02-06 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68664 --- Comment #18 from Segher Boessenkool --- Author: segher Date: Mon Feb 6 19:19:49 2017 New Revision: 245215 URL: https://gcc.gnu.org/viewcvs?rev=245215&root=gcc&view=rev Log: sched: Do not move expensive insns speculatively (PR68664) Schedul

[Bug rtl-optimization/68664] [6/7 Regression] Speculative sqrt in c-ray main loop causes large slow down

2017-02-06 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68664 Segher Boessenkool changed: What|Removed |Added Status|ASSIGNED|NEW --- Comment #19 from Segher Boe

[Bug rtl-optimization/68664] [6/7 Regression] Speculative sqrt in c-ray main loop causes large slow down

2017-02-06 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68664 --- Comment #22 from Segher Boessenkool --- (In reply to Jan Hubicka from comment #20) > There was also regression on cray for x86-64 > https://gcc.opensuse.org/c++bench-czerny/c-ray/ > Is it the same issue? I don't think so. But I don't know m

[Bug rtl-optimization/68664] [6/7 Regression] Speculative sqrt in c-ray main loop causes large slow down

2017-02-06 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68664 --- Comment #24 from Segher Boessenkool --- (In reply to Jan Hubicka from comment #23) > > > Also with profile feedback perhaps you have enough info to tell that the > > > speculative path is almost as likely as the original placement. > > > > M

[Bug rtl-optimization/64081] [5/6/7 Regression] r217827 prevents RTL loop unroll

2017-02-07 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64081 --- Comment #52 from Segher Boessenkool --- (In reply to Aldy Hernandez from comment #50) > Though at this > point, I'd rather we figure out why the erroneous code is being generated in > comment 45. If you can send me the output (.s and .c.*) w

[Bug rtl-optimization/64081] [5/6/7 Regression] r217827 prevents RTL loop unroll

2017-02-08 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64081 --- Comment #54 from Segher Boessenkool --- Thanks Aldy, this makes it clear what happens, and it is actually pretty simple. The patch changes check_simple_exit to also return true if check_complex_exit is true (as well as some other conditions)

[Bug translation/79397] AltiVec spelled incorrectly in rs6000.opt

2017-02-08 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79397 --- Comment #1 from Segher Boessenkool --- Author: segher Date: Wed Feb 8 09:59:55 2017 New Revision: 245276 URL: https://gcc.gnu.org/viewcvs?rev=245276&root=gcc&view=rev Log: rs6000: Fix spelling of AltiVec in rs6000.opt (PR79397) It was spel

[Bug translation/79397] AltiVec spelled incorrectly in rs6000.opt

2017-02-08 Thread segher at gcc dot gnu.org
||2017-02-08 CC||segher at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #2 from Segher Boessenkool --- Fixed on trunk; will commit to 5 and 6 in a bit. Thanks for the report.

[Bug rtl-optimization/64081] [5/6/7 Regression] r217827 prevents RTL loop unroll

2017-02-08 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64081 --- Comment #56 from Segher Boessenkool --- Yes, and a function that computes a "more permissive" version should not have "simple_loop" in its name, it is very misleading. Reusing existing functions to do something different may seem attractive

[Bug testsuite/79427] g++.dg/tls/thread_local-order2.C fails starting with r245249

2017-02-08 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79427 --- Comment #1 from Segher Boessenkool --- I get the correct output on BE (gcc110). This is glibc 2.18, maybe that is the difference?

[Bug rtl-optimization/64081] [5/6/7 Regression] r217827 prevents RTL loop unroll

2017-02-08 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64081 --- Comment #58 from Segher Boessenkool --- You can keep get_simple_loop_desc, find_simple_exit etc.; just make them inline functions or similar. I'm sceptical that this will not cause any more problems, we're deep into stage 4 already :-/

[Bug translation/79397] AltiVec spelled incorrectly in rs6000.opt

2017-02-08 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79397 --- Comment #3 from Segher Boessenkool --- Author: segher Date: Wed Feb 8 21:41:31 2017 New Revision: 245286 URL: https://gcc.gnu.org/viewcvs?rev=245286&root=gcc&view=rev Log: rs6000: Fix spelling of AltiVec in rs6000.opt (PR79397) It was spel

[Bug translation/79397] AltiVec spelled incorrectly in rs6000.opt

2017-02-08 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79397 --- Comment #4 from Segher Boessenkool --- Author: segher Date: Wed Feb 8 21:44:37 2017 New Revision: 245287 URL: https://gcc.gnu.org/viewcvs?rev=245287&root=gcc&view=rev Log: rs6000: Fix spelling of AltiVec in rs6000.opt (PR79397) It was spel

<    12   13   14   15   16   17   18   19   20   21   >