[Bug target/105325] power10: Error: operand out of range

2023-01-26 Thread acsawdey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105325 acsawdey at gcc dot gnu.org changed: What|Removed |Added CC||acsawdey at gcc dot

[Bug target/103197] ppc inline expansion of memcpy/memmove should not use lxsibzx/stxsibx for a single byte

2021-11-16 Thread acsawdey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103197 --- Comment #5 from acsawdey at gcc dot gnu.org --- Bisection reveals that this starts with this commit: 20d70cd2719815d9ea853314775ae5787648ece5 is the first bad commit commit 20d70cd2719815d9ea853314775ae5787648ece5 Author: Alan Modra Date

[Bug target/103197] ppc inline expansion of memcpy/memmove should not use lxsibzx/stxsibx for a single byte

2021-11-15 Thread acsawdey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103197 --- Comment #4 from acsawdey at gcc dot gnu.org --- I was compiling with -mcpu=power9, yes: /home2/sawdey/work/gcc/trunk/build/gcc/xgcc -B/home2/sawdey/work/gcc/trunk/build/gcc -O3 -mcpu=power9 bug2.c

[Bug target/103197] ppc inline expansion of memcpy/memmove should not use lxsibzx/stxsibx for a single byte

2021-11-11 Thread acsawdey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103197 --- Comment #2 from acsawdey at gcc dot gnu.org --- >From the reload dump: 0 Non input pseudo reload: reject++ 1 Non-pseudo reload: reject+=2 1 Non input pseudo reload: reject++ alt=0,overall

[Bug target/103197] ppc inline expansion of memcpy/memmove should not use lxsibzx/stxsibx for a single byte

2021-11-11 Thread acsawdey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103197 --- Comment #1 from acsawdey at gcc dot gnu.org --- Looking at trunk, after expand we have this: (note 5 1 2 2 [bb 2] NOTE_INSN_BASIC_BLOCK) (insn 2 5 3 2 (set (reg/v/f:DI 117 [ a ]) (reg:DI 3 3 [ a ])) "bug2.c":3:1 -1 (n

[Bug target/103197] New: ppc inline expansion of memcpy/memmove should not use lxsibzx/stxsibx for a single byte

2021-11-11 Thread acsawdey at gcc dot gnu.org via Gcc-bugs
Severity: normal Priority: P3 Component: target Assignee: unassigned at gcc dot gnu.org Reporter: acsawdey at gcc dot gnu.org Target Milestone: --- This got broken sometime in gcc 10 timeframe. For this test case: #include void m(char *a, char *b

[Bug target/100996] rs6000 p10 vector add-add fusion should work with -m32 but doesn't

2021-06-09 Thread acsawdey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100996 acsawdey at gcc dot gnu.org changed: What|Removed |Added Ever confirmed|0 |1 Target

[Bug target/100996] New: rs6000 p10 vector add-add fusion should work with -m32 but doesn't

2021-06-09 Thread acsawdey at gcc dot gnu.org via Gcc-bugs
ormal Priority: P3 Component: target Assignee: unassigned at gcc dot gnu.org Reporter: acsawdey at gcc dot gnu.org Target Milestone: --- The fusion-p10-addadd.c test case does not get vector add-add fusion when compiling with -m32: /home/sawdey/work/gcc/trunk/

[Bug target/97926] ICE in patch_jump_insn, at cfgrtl.c:1298

2021-03-18 Thread acsawdey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97926 --- Comment #3 from acsawdey at gcc dot gnu.org --- So the underlying problem here is that the unordered comparisons are not allowed with -ffinite-math-only due to this predicate: ;; Return 1 if OP is a comparison operation that is valid for a

[Bug target/97926] ICE in patch_jump_insn, at cfgrtl.c:1298

2021-03-16 Thread acsawdey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97926 --- Comment #2 from acsawdey at gcc dot gnu.org --- patch_jump_insn() is running into a land mine -- the insn before modification is invalid: (gdb) p insn_invalid_p(insn, true) $4 = 1 (gdb) pr insn (jump_insn 18 17 114 6 (set (pc

[Bug target/99070] ICE in extract_constrain_insn, at recog.c:2670

2021-03-08 Thread acsawdey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99070 acsawdey at gcc dot gnu.org changed: What|Removed |Added Status|NEW |RESOLVED

[Bug target/99070] ICE in extract_constrain_insn, at recog.c:2670

2021-02-11 Thread acsawdey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99070 --- Comment #4 from acsawdey at gcc dot gnu.org --- OK, I see the fail with -mcpu=power9. Looks like I botched something with addressing and allowed D-form addresses when it should be DS-form. On power10 this would result in selection of a prefix

[Bug target/99070] ICE in extract_constrain_insn, at recog.c:2670

2021-02-11 Thread acsawdey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99070 acsawdey at gcc dot gnu.org changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |acsawdey at gcc dot

[Bug rtl-optimization/98692] Unitialized Values reported only with -Os

2021-01-18 Thread acsawdey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98692 --- Comment #7 from acsawdey at gcc dot gnu.org --- The inline expansion should be disabled by -Os, the patterns for cmpstr[n]si both have this: if (optimize_insn_for_size_p ()) FAIL;

[Bug target/98688] C++ modules support does not work on PowerPC with opaque MMA types vector_pair/vector_quad

2021-01-15 Thread acsawdey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98688 --- Comment #3 from acsawdey at gcc dot gnu.org --- Yeah it's pretty clear that something needs to be output, as with that code I get an error like this: In module imported at mma-module-2.C:1:1: mma_foo0: In function ‘int bar(__vector

[Bug target/98688] C++ modules support does not work on PowerPC with opaque MMA types vector_pair/vector_quad

2021-01-14 Thread acsawdey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98688 --- Comment #1 from acsawdey at gcc dot gnu.org --- I don't know if this is the right thing to do, but ignoring the opaque type here make the ICE go away. I suspect I need to construct a module test case using vector_pair/vector_quad to r

[Bug target/98688] New: C++ modules support does not work on PowerPC with opaque MMA types vector_pair/vector_quad

2021-01-14 Thread acsawdey at gcc dot gnu.org via Gcc-bugs
Severity: normal Priority: P3 Component: target Assignee: unassigned at gcc dot gnu.org Reporter: acsawdey at gcc dot gnu.org Target Milestone: --- Similar to PR98645, we run into trouble if we try to compile code using vector_pair/vector_quad using

[Bug c++/97947] [11 Regression] ICE in digest_init_r, at cp/typeck2.c:1145

2020-12-01 Thread acsawdey at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97947 acsawdey at gcc dot gnu.org changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |acsawdey at gcc dot

[Bug target/96791] ICE in convert_mode_scalar, at expr.c:412

2020-09-10 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96791 --- Comment #10 from acsawdey at gcc dot gnu.org --- For now, disabling use of POImode for expansion of memcpy/memmove to avoid this problem while we figure out the real fix: https://gcc.gnu.org/pipermail/gcc-patches/2020-September/553672.html

[Bug target/96791] ICE in convert_mode_scalar, at expr.c:412

2020-09-09 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96791 --- Comment #9 from acsawdey at gcc dot gnu.org --- I did post a small patch that fixes this, but more for the purpose of provoking discussion than because I am sure it is the right way to fix this. https://gcc.gnu.org/pipermail/gcc-patches/2020

[Bug target/96791] ICE in convert_mode_scalar, at expr.c:412

2020-09-02 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96791 --- Comment #8 from acsawdey at gcc dot gnu.org --- Another small test case, reduced from my compile failure of c/c-typeck.c and modified to provoke truncation from POImode to various other modes: typedef int *a; struct b { a ba; }; enum c { c1

[Bug target/96791] ICE in convert_mode_scalar, at expr.c:412

2020-08-31 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96791 --- Comment #7 from acsawdey at gcc dot gnu.org --- I wonder if this other case works properly when compiled with -m64. Trying to generate a stxvp with a 32-bit address seems odd.

[Bug target/96791] ICE in convert_mode_scalar, at expr.c:412

2020-08-27 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96791 acsawdey at gcc dot gnu.org changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |acsawdey at gcc dot

[Bug target/96791] ICE in convert_mode_scalar, at expr.c:412

2020-08-27 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96791 --- Comment #3 from acsawdey at gcc dot gnu.org --- This also requires -mbig which may be implicit in the original poster's build. But I see it failing as well.

[Bug target/96787] rs6000 mcpu=power10 miscompiles libiberty htab_delete() causing bootstrap failure

2020-08-25 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96787 --- Comment #3 from acsawdey at gcc dot gnu.org --- Never mind that, all I'm seeing is the lack of save/restore of r2 in the power10 version.

[Bug target/96787] rs6000 mcpu=power10 miscompiles libiberty htab_delete() causing bootstrap failure

2020-08-25 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96787 --- Comment #2 from acsawdey at gcc dot gnu.org --- I'm seeing some load-past-store code motion that happens when compiling for power10 vs power9 that makes me suspicious.

[Bug target/96787] rs6000 mcpu=power10 miscompiles libiberty htab_delete() causing bootstrap failure

2020-08-25 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96787 --- Comment #1 from acsawdey at gcc dot gnu.org --- Created attachment 49123 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=49123&action=edit hashtab.c with target power9 attribute on htab_delete()

[Bug target/96787] New: rs6000 mcpu=power10 miscompiles libiberty htab_delete() causing bootstrap failure

2020-08-25 Thread acsawdey at gcc dot gnu.org
Severity: normal Priority: P3 Component: target Assignee: unassigned at gcc dot gnu.org Reporter: acsawdey at gcc dot gnu.org Target Milestone: --- Created attachment 49122 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=49122&action=edit hashtab.c

[Bug c/96151] bootstrap fails due to ICE in c_omp_split_clauses

2020-07-10 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96151 --- Comment #1 from acsawdey at gcc dot gnu.org --- This compile is successful like this but fails if I add -mcpu=power9. /home2/sawdey/work/gcc/mamboCI/build-mambo/./prev-gcc/xg++ -B/home2/sawdey/work/gcc/mamboCI/build-mambo/./prev-gcc/ -B/opt

[Bug c/96151] New: bootstrap fails due to ICE in c_omp_split_clauses

2020-07-10 Thread acsawdey at gcc dot gnu.org
: c Assignee: unassigned at gcc dot gnu.org Reporter: acsawdey at gcc dot gnu.org Target Milestone: --- Started to see this on trunk last night. Tested again and still see it with r11-2018. configured with: /home2/sawdey/work/gcc/mamboCI/gcc-master/configure --prefix=/opt

[Bug target/95347] rs6000 mcpu=future generating stfs instead of pstfs for pc-relative references

2020-06-09 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95347 acsawdey at gcc dot gnu.org changed: What|Removed |Added Status|ASSIGNED|RESOLVED

[Bug target/95347] rs6000 mcpu=future generating stfs instead of pstfs for pc-relative references

2020-06-02 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95347 acsawdey at gcc dot gnu.org changed: What|Removed |Added Last reconfirmed||2020-06-02 Ever

[Bug target/95347] New: rs6000 mcpu=future generating stfs instead of pstfs for pc-relative references

2020-05-26 Thread acsawdey at gcc dot gnu.org
: normal Priority: P3 Component: target Assignee: unassigned at gcc dot gnu.org Reporter: acsawdey at gcc dot gnu.org Target Milestone: --- Problem exists in r11-639. /home2/sawdey/work/gcc/mamboCI/build-mambo/gcc/xgcc -B/home2/sawdey/work/gcc/mamboCI/build

[Bug target/94740] ICE on testsuite/gcc.dg/sso/t5.c with -mcpu=future -mpcrel -O1

2020-04-23 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94740 --- Comment #1 from acsawdey at gcc dot gnu.org --- Reduced test case: struct __attribute__((scalar_storage_order("big-endian"))) { int a; int b[]; } c; int d; int e() { d = c.b[0]; }

[Bug target/94740] New: ICE on testsuite/gcc.dg/sso/t5.c with -mcpu=future -mpcrel -O1

2020-04-23 Thread acsawdey at gcc dot gnu.org
Priority: P3 Component: target Assignee: unassigned at gcc dot gnu.org Reporter: acsawdey at gcc dot gnu.org Target Milestone: --- Compiler is trunk 3bcdb5dec72b6d7b197821c2b814bc9fc07f4628 on ppc64le power9 host. ~/work/gcc/trunk/build/gcc/xgcc -B/home2/sawdey/work

[Bug target/94622] testsuite/gcc.dg/atomic/c11-atomic-exec-1.c fails on powerpc64le with -mpcrel

2020-04-22 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94622 acsawdey at gcc dot gnu.org changed: What|Removed |Added Resolution|--- |FIXED Status

[Bug target/94622] testsuite/gcc.dg/atomic/c11-atomic-exec-1.c fails on powerpc64le with -mpcrel

2020-04-21 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94622 --- Comment #3 from acsawdey at gcc dot gnu.org --- I'm wondering if the same problem exists for atomic_store, store_quadpti, and pstq vs stq?

[Bug target/94622] testsuite/gcc.dg/atomic/c11-atomic-exec-1.c fails on powerpc64le with -mpcrel

2020-04-20 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94622 --- Comment #2 from acsawdey at gcc dot gnu.org --- Solution is going to be to always use plq if prefixed, which makes sense anyway for little endian because it avoids the ugly doubleword swap.

[Bug target/94622] testsuite/gcc.dg/atomic/c11-atomic-exec-1.c fails on powerpc64le with -mpcrel

2020-04-17 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94622 --- Comment #1 from acsawdey at gcc dot gnu.org --- Compiling with -dap we see: sync # 7[c=12 l=4] *hwsync plq 8,.LANCHOR0@pcrel# 8[c=8 l=12] load_quadpti mr 10,9 # 9[c=4 l=4

[Bug target/94622] testsuite/gcc.dg/atomic/c11-atomic-exec-1.c fails on powerpc64le with -mpcrel

2020-04-16 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94622 acsawdey at gcc dot gnu.org changed: What|Removed |Added Last reconfirmed||2020-04-16

[Bug target/94622] New: testsuite/gcc.dg/atomic/c11-atomic-exec-1.c fails on powerpc64le with -mpcrel

2020-04-16 Thread acsawdey at gcc dot gnu.org
: normal Priority: P3 Component: target Assignee: unassigned at gcc dot gnu.org Reporter: acsawdey at gcc dot gnu.org Target Milestone: --- Compile command: /home2/sawdey/work/gcc/mamboCI/build-mambo/gcc/xgcc -B/home2/sawdey/work/gcc/mamboCI/build-mambo/gcc

[Bug target/94542] test gcc/testsuite/gcc.dg/tls/pr24428-2.c generates incorrect code on ppc64le with -mpcrel -mcpu=future -O2

2020-04-09 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94542 acsawdey at gcc dot gnu.org changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |acsawdey at gcc dot

[Bug target/94542] New: test gcc/testsuite/gcc.dg/tls/pr24428-2.c generates incorrect code on ppc64le with -mpcrel -mcpu=future -O2

2020-04-09 Thread acsawdey at gcc dot gnu.org
Status: UNCONFIRMED Severity: normal Priority: P3 Component: target Assignee: unassigned at gcc dot gnu.org Reporter: acsawdey at gcc dot gnu.org Target Milestone: --- The test case is: __thread double thrtest[81]; int main () { double

[Bug target/92379] rs6000.c:5598:13: runtime error: shift exponent 64 is too large for 64-bit type 'long int'

2020-03-16 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92379 acsawdey at gcc dot gnu.org changed: What|Removed |Added Resolution|--- |FIXED Status

[Bug target/92379] rs6000.c:5598:13: runtime error: shift exponent 64 is too large for 64-bit type 'long int'

2020-03-06 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92379 acsawdey at gcc dot gnu.org changed: What|Removed |Added Status|NEW |ASSIGNED

[Bug target/93129] PPC memset not using vector instruction on >= Power8

2020-01-06 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93129 acsawdey at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last

[Bug target/93130] PPC simple memset not inlined

2020-01-06 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93130 acsawdey at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last

[Bug jit/87808] gcc_lib_dir is missing from libgccjit's search path when driver is not installed

2019-07-22 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87808 acsawdey at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed

[Bug rtl-optimization/88308] ICE in maybe_record_trace_start, at dwarf2cfi.c:2309

2019-02-15 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88308 acsawdey at gcc dot gnu.org changed: What|Removed |Added Status|NEW |RESOLVED

[Bug rtl-optimization/88308] ICE in maybe_record_trace_start, at dwarf2cfi.c:2309

2019-02-15 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88308 --- Comment #6 from acsawdey at gcc dot gnu.org --- Author: acsawdey Date: Fri Feb 15 15:41:25 2019 New Revision: 268942 URL: https://gcc.gnu.org/viewcvs?rev=268942&root=gcc&view=rev Log: 2019-02-15 Aaron Sawdey PR rtl-opti

[Bug target/88308] ICE in maybe_record_trace_start, at dwarf2cfi.c:2309

2019-02-13 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88308 --- Comment #5 from acsawdey at gcc dot gnu.org --- After some more digging, it appears that the problem is move_insn_for_shrink_wrap() is deleting and re-creating insns to move them from one BB to another. The label reference count gets

[Bug target/88308] ICE in maybe_record_trace_start, at dwarf2cfi.c:2309

2019-02-12 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88308 --- Comment #4 from acsawdey at gcc dot gnu.org --- Tracked down the difference between -m32 and -m64. In the -m64 case, rs6000_emit_move calls force_const_mem and that will set LABEL_PRESERVE_P on a label_ref that it finds, which is what marks

[Bug target/88308] ICE in maybe_record_trace_start, at dwarf2cfi.c:2309

2019-02-12 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88308 --- Comment #3 from acsawdey at gcc dot gnu.org --- One difference between compiling this -m32 and -m64 is that the label for the jump table is marked /s in the 64-bit version: (code_label/s 22 21 23 4 (nil) [4 uses]) (jump_table_data 23 22 24

[Bug target/89112] Incorrect code generated by rs6000 memcmp expansion

2019-02-09 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89112 acsawdey at gcc dot gnu.org changed: What|Removed |Added Status|ASSIGNED|RESOLVED

[Bug target/89112] Incorrect code generated by rs6000 memcmp expansion

2019-02-09 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89112 --- Comment #10 from acsawdey at gcc dot gnu.org --- Author: acsawdey Date: Sat Feb 9 17:11:06 2019 New Revision: 268725 URL: https://gcc.gnu.org/viewcvs?rev=268725&root=gcc&view=rev Log: 2019-02-09 Aaron Sawdey Backpor

[Bug target/89112] Incorrect code generated by rs6000 memcmp expansion

2019-02-05 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89112 --- Comment #9 from acsawdey at gcc dot gnu.org --- The fixes for this are in trunk now. I will backport to gcc-8-branch in a week and then this can be closed.

[Bug target/89112] Incorrect code generated by rs6000 memcmp expansion

2019-02-05 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89112 --- Comment #8 from acsawdey at gcc dot gnu.org --- Author: acsawdey Date: Tue Feb 5 16:32:06 2019 New Revision: 268547 URL: https://gcc.gnu.org/viewcvs?rev=268547&root=gcc&view=rev Log: 2019-02-05 Aaron Sawdey PR targ

[Bug target/89112] Incorrect code generated by rs6000 memcmp expansion

2019-02-05 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89112 --- Comment #7 from acsawdey at gcc dot gnu.org --- Author: acsawdey Date: Tue Feb 5 16:30:45 2019 New Revision: 268546 URL: https://gcc.gnu.org/viewcvs?rev=268546&root=gcc&view=rev Log: 2019-02-05 Aaron Sawdey PR targ

[Bug target/89112] Incorrect code generated by rs6000 memcmp expansion

2019-02-01 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89112 --- Comment #5 from acsawdey at gcc dot gnu.org --- This patch fixes the issue on trunk: Index: gcc/config/rs6000/rs6000.md === --- gcc/config/rs6000/rs6000.md (revision 268403

[Bug target/89112] Incorrect code generated by rs6000 memcmp expansion

2019-02-01 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89112 --- Comment #4 from acsawdey at gcc dot gnu.org --- Well I can't blame this one on the linker or optimization. The splitting for the case where the branch destination is too far is wrong in tf_: static char seq[96]; char

[Bug target/89112] Incorrect code generated by rs6000 memcmp expansion

2019-02-01 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89112 --- Comment #3 from acsawdey at gcc dot gnu.org --- It appears that gcc decided to split the bdnzt generated by the memcmp expansion because the destination was out of range, and produced this: bdz $+12 beq 0,$+8 b $+8;b

[Bug target/89112] Incorrect code generated by rs6000 memcmp expansion

2019-01-30 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89112 --- Comment #2 from acsawdey at gcc dot gnu.org --- I'm seeing this on both gcc-8-branch and trunk, but only with -mcpu=power9. I'll figure out what happened here and get it fixed in trunk then back ported to 8.

[Bug target/89112] Incorrect code generated by rs6000 memcmp expansion

2019-01-30 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89112 acsawdey at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last

[Bug target/88027] PowerPC generates slightly weird code for memset

2019-01-04 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88027 acsawdey at gcc dot gnu.org changed: What|Removed |Added Status|ASSIGNED|RESOLVED

[Bug target/88027] PowerPC generates slightly weird code for memset

2019-01-03 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88027 acsawdey at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last

[Bug target/88027] PowerPC generates slightly weird code for memset

2018-11-14 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88027 --- Comment #3 from acsawdey at gcc dot gnu.org --- This appears to have to do with alignment. In this test case, expand_block_clear() sees alignment of only 8 bits for the pointer p. If you declare a local struct st and pass that to

[Bug target/88027] PowerPC generates slightly weird code for memset

2018-11-14 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88027 acsawdey at gcc dot gnu.org changed: What|Removed |Added CC||acsawdey at gcc dot gnu.org

[Bug target/87474] ICE in extract_insn, at recog.c:2305

2018-10-02 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87474 acsawdey at gcc dot gnu.org changed: What|Removed |Added Status|ASSIGNED|RESOLVED

[Bug target/87474] ICE in extract_insn, at recog.c:2305

2018-10-02 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87474 --- Comment #4 from acsawdey at gcc dot gnu.org --- Author: acsawdey Date: Tue Oct 2 17:31:53 2018 New Revision: 264799 URL: https://gcc.gnu.org/viewcvs?rev=264799&root=gcc&view=rev Log: 2018-10-02 Aaron Sawdey PR targ

[Bug target/87474] ICE in extract_insn, at recog.c:2305

2018-10-01 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87474 acsawdey at gcc dot gnu.org changed: What|Removed |Added Status|NEW |ASSIGNED

[Bug target/86222] ICE in final_scan_insn_1 calling strncmp() with a bound of PTRDIFF_MAX + 1

2018-06-26 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86222 acsawdey at gcc dot gnu.org changed: What|Removed |Added Status|ASSIGNED|RESOLVED

[Bug target/86222] ICE in final_scan_insn_1 calling strncmp() with a bound of PTRDIFF_MAX + 1

2018-06-26 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86222 --- Comment #6 from acsawdey at gcc dot gnu.org --- Author: acsawdey Date: Tue Jun 26 16:43:38 2018 New Revision: 262157 URL: https://gcc.gnu.org/viewcvs?rev=262157&root=gcc&view=rev Log: 2018-06-26 Aaron Sawdey Backport fr

[Bug target/86222] ICE in final_scan_insn_1 calling strncmp() with a bound of PTRDIFF_MAX + 1

2018-06-22 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86222 --- Comment #5 from acsawdey at gcc dot gnu.org --- Author: acsawdey Date: Fri Jun 22 15:37:36 2018 New Revision: 261906 URL: https://gcc.gnu.org/viewcvs?rev=261906&root=gcc&view=rev Log: Forgot PR target/86222 in ChangeLog Modified:

[Bug target/86222] ICE in final_scan_insn_1 calling strncmp() with a bound of PTRDIFF_MAX + 1

2018-06-21 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86222 --- Comment #4 from acsawdey at gcc dot gnu.org --- Well when compiling this with -m32 -mcpu=power[6789] I get this for the rtx of the length argument: (const_int -2147483648 [0x8000]) So when I am doing UINTVAL (bytes_rtx) I get

[Bug target/86222] ICE in final_scan_insn_1 calling strncmp() with a bound of PTRDIFF_MAX + 1

2018-06-21 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86222 --- Comment #3 from acsawdey at gcc dot gnu.org --- OK, so this requires -m32 and also -mcpu=power6 or higher. I have reproduced it so should have a fix shortly.

[Bug target/86222] ICE in final_scan_insn_1 calling strncmp() with a bound of PTRDIFF_MAX + 1

2018-06-21 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86222 acsawdey at gcc dot gnu.org changed: What|Removed |Added Status|NEW |ASSIGNED

[Bug target/83660] ICE with vec_extract inside expression statement

2018-04-23 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83660 acsawdey at gcc dot gnu.org changed: What|Removed |Added Status|REOPENED|RESOLVED

[Bug target/83660] ICE with vec_extract inside expression statement

2018-04-23 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83660 --- Comment #18 from acsawdey at gcc dot gnu.org --- Author: acsawdey Date: Tue Apr 24 00:19:43 2018 New Revision: 259590 URL: https://gcc.gnu.org/viewcvs?rev=259590&root=gcc&view=rev Log: 2018-04-23 Aaron Sawdey Backp

[Bug target/83660] ICE with vec_extract inside expression statement

2018-04-23 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83660 --- Comment #17 from acsawdey at gcc dot gnu.org --- Author: acsawdey Date: Tue Apr 24 00:14:21 2018 New Revision: 259586 URL: https://gcc.gnu.org/viewcvs?rev=259586&root=gcc&view=rev Log: 2018-04-23 Aaron Sawdey Backp

[Bug target/85436] [7 Regression] ICE compiling go code with -mcpu=power9

2018-04-17 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85436 --- Comment #1 from acsawdey at gcc dot gnu.org --- Created attachment 43966 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43966&action=edit shorter reduced test case I've further reduced the test case and now it's on

[Bug target/85436] New: [7 Regression] ICE compiling go code with -mcpu=power9

2018-04-17 Thread acsawdey at gcc dot gnu.org
Component: target Assignee: unassigned at gcc dot gnu.org Reporter: acsawdey at gcc dot gnu.org CC: bergner at gcc dot gnu.org, segher at gcc dot gnu.org, wschmidt at gcc dot gnu.org Target Milestone: --- Target: powerpc64le-linux-gnu

[Bug target/83660] ICE with vec_extract inside expression statement

2018-04-16 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83660 acsawdey at gcc dot gnu.org changed: What|Removed |Added Status|RESOLVED|REOPENED

[Bug target/83660] ICE with vec_extract inside expression statement

2018-04-16 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83660 acsawdey at gcc dot gnu.org changed: What|Removed |Added Status|ASSIGNED|RESOLVED

[Bug target/83660] ICE with vec_extract inside expression statement

2018-04-16 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83660 --- Comment #14 from acsawdey at gcc dot gnu.org --- Author: acsawdey Date: Mon Apr 16 14:50:06 2018 New Revision: 259403 URL: https://gcc.gnu.org/viewcvs?rev=259403&root=gcc&view=rev Log: 2018-04-16 Aaron Sawdey PR targ

[Bug target/83660] ICE with vec_extract inside expression statement

2018-04-13 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83660 --- Comment #12 from acsawdey at gcc dot gnu.org --- This function is called from cp/semantics.c maybe_cleanup_point_expr() tree fold_build_cleanup_point_expr (tree type, tree expr) { /* If the expression does not have side effects then we

[Bug target/83660] ICE with vec_extract inside expression statement

2018-04-12 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83660 acsawdey at gcc dot gnu.org changed: What|Removed |Added Status|NEW |ASSIGNED

[Bug target/83660] ICE with vec_extract inside expression statement

2018-04-12 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83660 acsawdey at gcc dot gnu.org changed: What|Removed |Added CC||acsawdey at gcc dot gnu.org

[Bug target/85321] Missing documentation and option misc for ppc64le

2018-04-11 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85321 acsawdey at gcc dot gnu.org changed: What|Removed |Added Status|ASSIGNED|RESOLVED

[Bug target/85321] Missing documentation and option misc for ppc64le

2018-04-11 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85321 --- Comment #5 from acsawdey at gcc dot gnu.org --- Author: acsawdey Date: Wed Apr 11 15:25:42 2018 New Revision: 259324 URL: https://gcc.gnu.org/viewcvs?rev=259324&root=gcc&view=rev Log: 2018-04-11 Aaron Sawdey PR targ

[Bug target/85321] Missing documentation and option misc for ppc64le

2018-04-10 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85321 --- Comment #4 from acsawdey at gcc dot gnu.org --- Author: acsawdey Date: Tue Apr 10 22:05:41 2018 New Revision: 259302 URL: https://gcc.gnu.org/viewcvs?rev=259302&root=gcc&view=rev Log: 2018-04-10 Aaron Sawdey PR targ

[Bug target/83822] trunk/gcc/config/rs6000/rs6000-string.c:970]: (style) Redundant condition

2018-04-02 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83822 acsawdey at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED

[Bug target/83822] trunk/gcc/config/rs6000/rs6000-string.c:970]: (style) Redundant condition

2018-03-30 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83822 --- Comment #4 from acsawdey at gcc dot gnu.org --- Author: acsawdey Date: Fri Mar 30 12:17:31 2018 New Revision: 258975 URL: https://gcc.gnu.org/viewcvs?rev=258975&root=gcc&view=rev Log: 2018-03-30 Aaron Sawdey PR targ

[Bug target/83707] g++.dg/eh/simd-3.C fails on power7 -m32

2018-03-29 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83707 acsawdey at gcc dot gnu.org changed: What|Removed |Added Status|NEW |RESOLVED

[Bug target/83707] g++.dg/eh/simd-3.C fails on power7 -m32

2018-03-29 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83707 --- Comment #5 from acsawdey at gcc dot gnu.org --- I can also confirm with trunk 258957 I do not see this fail with -m32 -mcpu=power7.

[Bug target/84743] default widths for parallel reassociation now hurt rather than help

2018-03-13 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84743 acsawdey at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED

[Bug target/84743] default widths for parallel reassociation now hurt rather than help

2018-03-13 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84743 --- Comment #5 from acsawdey at gcc dot gnu.org --- Author: acsawdey Date: Tue Mar 13 16:28:09 2018 New Revision: 258495 URL: https://gcc.gnu.org/viewcvs?rev=258495&root=gcc&view=rev Log: 2018-03-13 Aaron Sawdey PR targ

[Bug target/84743] default widths for parallel reassociation now hurt rather than help

2018-03-09 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84743 acsawdey at gcc dot gnu.org changed: What|Removed |Added Priority|P1 |P3 --- Comment #4 from

[Bug target/84743] default widths for parallel reassociation now hurt rather than help

2018-03-07 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84743 --- Comment #3 from acsawdey at gcc dot gnu.org --- Yes I'm digging into this now and omnetpp is at the top of the list. I can see if there is a difference between cpu2006 and 2017 as well. For gcc7 I used 2006 to determine the widths.

[Bug target/84743] New: default widths for parallel reassociation now hurt rather than help

2018-03-06 Thread acsawdey at gcc dot gnu.org
Priority: P3 Component: target Assignee: acsawdey at gcc dot gnu.org Reporter: acsawdey at gcc dot gnu.org Target Milestone: --- Target: powerpc64le power8 The width settings in rs6000_reassociation_width() were chosen to help performance for SPEC CPU

[Bug middle-end/84433] gcc 7 and before miscompile loop and remove exit due to incorrect range calculation

2018-02-19 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84433 --- Comment #8 from acsawdey at gcc dot gnu.org --- It looks like both gcc 7 and 8 assume that the statement ptrA->sA[ptrB->int1].zt = parm1; will only be executed 14+1 times because of the declaration sA[15]. However gcc 7 assum

  1   2   3   >