[Bug ipa/99122] [10/11 Regression] ICE in force_constant_size, at gimplify.c:733

2021-02-18 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99122 --- Comment #13 from Martin Jambor --- I think that you want to disable inlining in the case when the callee has a formal parameter which is a VLA (as opposed to a VLA actual argument of a call), probably in inline_forbidden_p. When just the

[Bug ipa/99122] [10/11 Regression] ICE in force_constant_size, at gimplify.c:733

2021-02-17 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99122 --- Comment #6 from Martin Jambor --- (In reply to Jakub Jelinek from comment #5) > That could perhaps work for the #c0 testcase where the function actually has > a non-VL parameter and so garbage in garbage out. > But would that work also for

[Bug ipa/99122] [10/11 Regression] ICE in force_constant_size, at gimplify.c:733

2021-02-17 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99122 --- Comment #4 from Martin Jambor --- With the patch from comment #3, the following sequence with the problematic call: x.1_26 = __builtin_alloca_with_align (_24, 8); g (WITH_SIZE_EXPR <*x.1_26, _22>, WITH_SIZE_EXPR <*x.1_26, _22>);

[Bug ipa/99122] [10/11 Regression] ICE in force_constant_size, at gimplify.c:733

2021-02-17 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99122 --- Comment #3 from Martin Jambor --- Looking at how expr.c deals with WITH_SIZE_EXPR, perhaps we should do something like the following: diff --git a/gcc/tree-inline.c b/gcc/tree-inline.c index a710fa59027..cdabeb6bafd 100644 ---

[Bug debug/95343] IPA-SRA can result in wrong debug info about removed function arguments

2021-02-17 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95343 Martin Jambor changed: What|Removed |Added Resolution|--- |FIXED Status|ASSIGNED

[Bug target/99083] Big run-time regressions of 519.lbm_r with LTO

2021-02-15 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99083 --- Comment #9 from Martin Jambor --- I will benchmark the patch later this week, just so that we know, but I agree that reverting the patch and applying it again at the beginning of stage1 is probably the best.

[Bug target/99083] New: Big run-time regressions of 519.lbm_r with LTO

2021-02-12 Thread jamborm at gcc dot gnu.org via Gcc-bugs
: target Assignee: unassigned at gcc dot gnu.org Reporter: jamborm at gcc dot gnu.org CC: ubizjak at gmail dot com Blocks: 26163 Target Milestone: --- Host: x86_64-linux Target: x86_64-linux On AMD Zen2 CPUs, 519.lbm_r is 62.12

[Bug middle-end/26163] [meta-bug] missed optimization in SPEC (2k17, 2k and 2k6 and 95)

2021-02-04 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=26163 Bug 26163 depends on bug 90234, which changed state. Bug 90234 Summary: 503.bwaves_r is 6% slower on Zen1/Zen2 CPUs at -Ofast with native march/mtune than with generic ones https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90234 What

[Bug target/90234] 503.bwaves_r is 6% slower on Zen1/Zen2 CPUs at -Ofast with native march/mtune than with generic ones

2021-02-04 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90234 Martin Jambor changed: What|Removed |Added Resolution|--- |FIXED Status|UNCONFIRMED

[Bug middle-end/26163] [meta-bug] missed optimization in SPEC (2k17, 2k and 2k6 and 95)

2021-02-04 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=26163 Bug 26163 depends on bug 94375, which changed state. Bug 94375 Summary: 548.exchange2_r run time is 8-18% worse than GCC 9 at -Ofast -march=native https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94375 What|Removed

[Bug tree-optimization/53947] [meta-bug] vectorizer missed-optimizations

2021-02-04 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=53947 Bug 53947 depends on bug 94375, which changed state. Bug 94375 Summary: 548.exchange2_r run time is 8-18% worse than GCC 9 at -Ofast -march=native https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94375 What|Removed

[Bug tree-optimization/94375] 548.exchange2_r run time is 8-18% worse than GCC 9 at -Ofast -march=native

2021-02-04 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94375 Martin Jambor changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug target/94373] 548.exchange2_r run time is 16-35% worse than GCC 9 at -O2 and generic march/mtune

2021-02-04 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94373 Martin Jambor changed: What|Removed |Added See Also||https://gcc.gnu.org/bugzill

[Bug middle-end/26163] [meta-bug] missed optimization in SPEC (2k17, 2k and 2k6 and 95)

2021-02-04 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=26163 Bug 26163 depends on bug 94400, which changed state. Bug 94400 Summary: 531.deepsjeng_r is 7% slower at -O2 -march=znver2 than GCC 9 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94400 What|Removed |Added

[Bug target/94400] 531.deepsjeng_r is 7% slower at -O2 -march=znver2 than GCC 9

2021-02-04 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94400 Martin Jambor changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug tree-optimization/97588] Overzealous SRA of boolean bitfields

2021-01-22 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97588 Martin Jambor changed: What|Removed |Added Ever confirmed|0 |1 Status|UNCONFIRMED

[Bug tree-optimization/98255] [10/11 Regression] wrong code at -Os and above with -fPIC on x86_64-pc-linux-gnu

2021-01-21 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98255 --- Comment #10 from Martin Jambor --- OK, adding an additional check whether tree_could_trap_p is of course easy. I'll wait a little while if the discussion about get_ref_base_and_extent perhaps leads to a different solution but if not, I will

[Bug tree-optimization/98255] [10/11 Regression] wrong code at -Os and above with -fPIC on x86_64-pc-linux-gnu

2021-01-21 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98255 --- Comment #7 from Martin Jambor --- Even our constant folding thinks the unsigned expression wraps around. If I tell SRA to fold the expression if the base is a string_cst, the invalid dereference is avoided. My experiment was (I am not

[Bug tree-optimization/98255] [10/11 Regression] wrong code at -Os and above with -fPIC on x86_64-pc-linux-gnu

2021-01-21 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98255 --- Comment #5 from Martin Jambor --- Right, the issue is that SRA depends on get_ref_base_and_extent to figure out what is being accessed (and so whether it is safe) and that function believes the load is safely from within the array.

[Bug tree-optimization/98255] [10/11 Regression] wrong code at -Os and above with -fPIC on x86_64-pc-linux-gnu

2021-01-21 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98255 --- Comment #3 from Martin Jambor --- So SRA sees statements: n[0][2] = "\t\x02\b"; and later _11 = n[0][3][4294967294]; The latter loads a scalar sitting inside what the store above initialized (according to get_ref_base_and_extent) and so

[Bug ipa/98078] ICE in cgraph_add_edge_to_call_site_hash, at cgraph.c:698 since r6-1705-gd88511aec7338a93

2021-01-21 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98078 --- Comment #4 from Martin Jambor --- Actually no, that would be papering over a bigger problem. After looking at the issue a bit more, I proposed a patch on the mailing list: https://gcc.gnu.org/pipermail/gcc-patches/2021-January/563962.html

[Bug c++/98744] [11 Regression] ICE in gimple_call_arg, at gimple.h:3246 since r11-6735-g424deca72b63e644

2021-01-20 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98744 Martin Jambor changed: What|Removed |Added Component|ipa |c++ --- Comment #3 from Martin Jambor

[Bug ipa/98744] [11 Regression] ICE in gimple_call_arg, at gimple.h:3246 since r11-6735-g424deca72b63e644

2021-01-20 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98744 --- Comment #2 from Martin Jambor --- That cannot be the problem. IPA-SRA re-creates the call statements and builds them with gimple_build_call_vec (callee_decl, vargs); where calle_decl is the new function which has had its type adjusted and

[Bug ipa/98690] [10/11 Regression] unexpected "'removed_return.213' may be used uninitialized in this function" causes crash since r10-3311-gff6686d2e5f797d6

2021-01-18 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98690 --- Comment #3 from Martin Jambor --- I have proposed a patch on the mailing list: https://gcc.gnu.org/pipermail/gcc-patches/2021-January/563790.html

[Bug ipa/98078] ICE in cgraph_add_edge_to_call_site_hash, at cgraph.c:698 since r6-1705-gd88511aec7338a93

2021-01-18 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98078 --- Comment #3 from Martin Jambor --- Here is what happens. An IPA-CP clone for a particular devirtualziation context is created but all devirtualziations based on it are speculative. Then the clone is inlined at one of its call sites and the

[Bug ipa/98222] [11 Regression] ICE at -O3 on x86_64-pc-linux-gnu: verify_cgraph_node failed since r11-4267-g0e590b68fa374365

2021-01-17 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98222 Martin Jambor changed: What|Removed |Added Resolution|--- |FIXED Status|ASSIGNED

[Bug ipa/98222] [11 Regression] ICE at -O3 on x86_64-pc-linux-gnu: verify_cgraph_node failed since r11-4267-g0e590b68fa374365

2021-01-13 Thread jamborm at gcc dot gnu.org via Gcc-bugs
at gcc dot gnu.org |jamborm at gcc dot gnu.org --- Comment #2 from Martin Jambor --- I'll have a look

[Bug tree-optimization/58243] Suboptimal structure initialization with tree-sra

2021-01-08 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=58243 Martin Jambor changed: What|Removed |Added Resolution|--- |FIXED Status|UNCONFIRMED

[Bug tree-optimization/47059] compiler fails to coalesce loads/stores

2021-01-08 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=47059 Martin Jambor changed: What|Removed |Added Resolution|--- |FIXED Status|UNCONFIRMED

[Bug tree-optimization/47059] compiler fails to coalesce loads/stores

2021-01-08 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=47059 Martin Jambor changed: What|Removed |Added CC||jamborm at gcc dot gnu.org --- Comment

[Bug tree-optimization/36602] memset should be optimized into an empty CONSTRUCTOR

2021-01-08 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=36602 Martin Jambor changed: What|Removed |Added CC||jamborm at gcc dot gnu.org --- Comment

[Bug target/80689] 128 bit loads generated for structure copying with gcc 7.1.0 and leads to STLF stalls in avx2 targets.

2021-01-08 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80689 Martin Jambor changed: What|Removed |Added Status|ASSIGNED|NEW Assignee|jamborm at gcc

[Bug lto/45375] [meta-bug] Issues with building Mozilla (i.e. Firefox) with LTO

2021-01-07 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=45375 Bug 45375 depends on bug 45791, which changed state. Bug 45791 Summary: Missed devirtualization https://gcc.gnu.org/bugzilla/show_bug.cgi?id=45791 What|Removed |Added

[Bug tree-optimization/45791] Missed devirtualization

2021-01-07 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=45791 Martin Jambor changed: What|Removed |Added Resolution|--- |FIXED Status|NEW

[Bug lto/50430] Constructors of static external vars are throwed away leading to missed optimizations (and ipa-cp ICE).

2021-01-07 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=50430 --- Comment #5 from Martin Jambor --- Am I correct thinking that this has been addressed (long time ago)? The entire optimized dump of the testcase from comment #3 is now the following, so no missed devirtualization there: void

[Bug ipa/79966] [9/10/11 Regression] Test gfortran.dg/pr79966.f90 slow again, inliner hits max-inline-insns-auto

2021-01-07 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79966 Martin Jambor changed: What|Removed |Added Summary|[9/10/11 Regression] run|[9/10/11 Regression] Test

[Bug ipa/79966] [9/10/11 Regression] run time more than twice slower when using -fipa-cp-clone

2021-01-07 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79966 --- Comment #13 from Martin Jambor --- I can confirm this, even on current trunk. The reason is that runtptests/3 -> tp_sum/5 is not inlined because it exceeds max-inline-insns-auto. I have to set the param to 43 - the default is 15 - for the

[Bug middle-end/26163] [meta-bug] missed optimization in SPEC (2k17, 2k and 2k6 and 95)

2020-12-07 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=26163 Bug 26163 depends on bug 97816, which changed state. Bug 97816 Summary: [11 Regression] ICE in good_cloning_opportunity_p, at ipa-cp.c:3266 since r11-4949-gb86aedb0cc083efe712e530a723f1237051a6b56

[Bug ipa/97816] [11 Regression] ICE in good_cloning_opportunity_p, at ipa-cp.c:3266 since r11-4949-gb86aedb0cc083efe712e530a723f1237051a6b56

2020-12-07 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97816 Martin Jambor changed: What|Removed |Added Resolution|--- |FIXED Status|ASSIGNED

[Bug middle-end/26163] [meta-bug] missed optimization in SPEC (2k17, 2k and 2k6 and 95)

2020-12-01 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=26163 Bug 26163 depends on bug 94406, which changed state. Bug 94406 Summary: 503.bwaves_r is 11% slower on Zen2 CPUs than GCC 9 with -Ofast -march=native https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94406 What|Removed

[Bug tree-optimization/94406] 503.bwaves_r is 11% slower on Zen2 CPUs than GCC 9 with -Ofast -march=native

2020-12-01 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94406 Martin Jambor changed: What|Removed |Added Resolution|--- |FIXED Status|UNCONFIRMED

[Bug ipa/93385] [10/11 Regression] wrong code with u128 modulo at -O2 -fno-dce -fno-ipa-cp -fno-tree-dce

2020-11-26 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93385 --- Comment #38 from Martin Jambor --- *** Bug 97980 has been marked as a duplicate of this bug. ***

[Bug tree-optimization/97980] [10/11 Regression] wrong code with "-O3 -fno-dce -fno-inline-functions-called-once -fno-inline-small-functions -fno-tree-ccp -fno-tree-dce -fno-tree-vrp" since r10-3311-g

2020-11-26 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97980 Martin Jambor changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug ipa/97816] [11 Regression] ICE in good_cloning_opportunity_p, at ipa-cp.c:3266 since r11-4949-gb86aedb0cc083efe712e530a723f1237051a6b56

2020-11-13 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97816 --- Comment #5 from Martin Jambor --- As noted in the commit message above, the ICE will go away but the underlying issue stays so please keep this opened until I fix it, hopefully no later than next week.

[Bug ipa/97816] [11 Regression] ICE in good_cloning_opportunity_p, at ipa-cp.c:3266 since r11-4949-gb86aedb0cc083efe712e530a723f1237051a6b56

2020-11-13 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97816 Martin Jambor changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |jamborm at gcc dot gnu.org

[Bug ipa/97695] [11 Regression] wrong code at -O3 on x86_64-pc-linux-gnu since r11-4587-gae7a23a3fab74.

2020-11-03 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97695 --- Comment #7 from Martin Jambor --- (In reply to Jan Hubicka from comment #5) > I see you have patch, too :) > However we do not want to copy clone info to every inline clone (since > the body is materialized just once). The problem is that

[Bug ipa/97695] [11 Regression] wrong code at -O3 on x86_64-pc-linux-gnu since r11-4587-gae7a23a3fab74.

2020-11-03 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97695 --- Comment #4 from Martin Jambor --- And the reason is not copying tree_map in cgraph_node::create_clone (when called from clone_inlined_nodes). The following should fix it. In theory we need a mechanism for create_virtual_clone to

[Bug ipa/97695] [11 Regression] wrong code at -O3 on x86_64-pc-linux-gnu since r11-4587-gae7a23a3fab74.

2020-11-03 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97695 --- Comment #3 from Martin Jambor --- It is a clone materialization problem. IPA-CP clones f.part.0 twice and the second time tree_function_versioning receives NULL tree_map.

[Bug c/97578] ice during IPA pass: inline

2020-11-02 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97578 Martin Jambor changed: What|Removed |Added CC||jamborm at gcc dot gnu.org --- Comment

[Bug tree-optimization/97456] [10/11 Regression] An incorrect optimization causes a function to always return the same value when using -flto

2020-10-26 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97456 Martin Jambor changed: What|Removed |Added Resolution|--- |FIXED Status|NEW

[Bug tree-optimization/97456] [10/11 Regression] An incorrect optimization causes a function to always return the same value when using -flto

2020-10-16 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97456 Martin Jambor changed: What|Removed |Added Component|ipa |tree-optimization --- Comment #7 from

[Bug ipa/97456] [10/11 Regression] An incorrect optimization causes a function to always return the same value when using -flto

2020-10-16 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97456 --- Comment #6 from Martin Jambor --- I have posted the patch to the mailing list: https://gcc.gnu.org/pipermail/gcc-patches/2020-October/556399.html

[Bug ipa/97456] [10/11 Regression] An incorrect optimization causes a function to always return the same value when using -flto

2020-10-16 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97456 --- Comment #5 from Martin Jambor --- And indeed the following avoids the issue: diff --git a/gcc/tree-complex.c b/gcc/tree-complex.c index 2e54bbb917c..71ad7c18523 100644 --- a/gcc/tree-complex.c +++ b/gcc/tree-complex.c @@ -570,8 +570,10 @@

[Bug ipa/97456] [10/11 Regression] An incorrect optimization causes a function to always return the same value when using -flto

2020-10-16 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97456 --- Comment #4 from Martin Jambor --- Looking at Martin's reduced testcase (In reply to Richard Biener from comment #2) > Confirmed with -fwhole-program -O3 IPA SRA messes things up here cloning > wrong > and producing the strange > >

[Bug ipa/96394] [10/11 Regression] ICE in add_new_edges_to_heap, at ipa-inline.c:1746 (-O3 PGO)

2020-10-16 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96394 Martin Jambor changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug tree-optimization/96818] [11 Regression] ICE: in decompose, at wide-int.h:984 at -O since r11-2883

2020-10-14 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96818 --- Comment #10 from Martin Jambor --- Is this bug still "WAITING" for something?

[Bug bootstrap/97319] New: LTO profiledbootstrap (C/C++/Fotran only) fails with a segfault in selftest

2020-10-07 Thread jamborm at gcc dot gnu.org via Gcc-bugs
: normal Priority: P3 Component: bootstrap Assignee: unassigned at gcc dot gnu.org Reporter: jamborm at gcc dot gnu.org CC: hubicka at gcc dot gnu.org Target Milestone: --- Host: x86_64-linux Target: x86_64-linux Since

[Bug ipa/96394] [10/11 Regression] ICE in add_new_edges_to_heap, at ipa-inline.c:1746 (-O3 PGO)

2020-10-01 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96394 --- Comment #18 from Martin Jambor --- I proposed the patch on the mailing list (I guess I should put Martin's name at least to the testsuite ChangeLog and probably to both): https://gcc.gnu.org/pipermail/gcc-patches/2020-October/555284.html

[Bug ipa/96394] [10/11 Regression] ICE in add_new_edges_to_heap, at ipa-inline.c:1746 (-O3 PGO)

2020-09-25 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96394 --- Comment #15 from Martin Jambor --- so after Martin asked some good questions, it turns out this should probably be avoided in ipa-prop, after all, as with, for example (untested): diff --git a/gcc/ipa-prop.c b/gcc/ipa-prop.c index

[Bug ipa/96394] [10/11 Regression] ICE in add_new_edges_to_heap, at ipa-inline.c:1746 (-O3 PGO)

2020-09-25 Thread jamborm at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96394 --- Comment #14 from Martin Jambor --- I can confirm the analysis, except that I see the edge we're trying to add to the heap as already inlined (as a speculative edge it got inlined even its caller was). Also just not adding an edge with

[Bug tree-optimization/96820] ICE in verify_sra_access_forest with array and out of bounds reference

2020-09-04 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96820 Martin Jambor changed: What|Removed |Added Resolution|--- |FIXED Status|ASSIGNED

[Bug tree-optimization/96820] ICE in verify_sra_access_forest with array and out of bounds reference

2020-08-30 Thread jamborm at gcc dot gnu.org
at gcc dot gnu.org |jamborm at gcc dot gnu.org --- Comment #6 from Martin Jambor --- I proposed a fix on the mailing list: https://gcc.gnu.org/pipermail/gcc-patches/2020-August/552900.html

[Bug tree-optimization/96820] ICE on x86_64-linux-gnu with `-m32` and from `-O0` to `-O3` (internal compiler error: in verify_sra_access_forest, at tree-sra.c:2358)

2020-08-28 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96820 --- Comment #5 from Martin Jambor --- (In reply to Chengnian Sun from comment #1) > Not sure if this is a dup to > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96730 No, this time it's build_user_friendly_ref_for_offset turning a nonsensical

[Bug tree-optimization/96730] [10/11 Regression] ICE on x86_64-linux-gnu with `-O1` to `-O3` (in verify_sra_access_forest, at tree-sra.c:2352) since r10-6320-g5b9e89c922dc2e7e

2020-08-25 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96730 Martin Jambor changed: What|Removed |Added Resolution|--- |FIXED Status|ASSIGNED

[Bug tree-optimization/96730] [10/11 Regression] ICE on x86_64-linux-gnu with `-O1` to `-O3` (in verify_sra_access_forest, at tree-sra.c:2352) since r10-6320-g5b9e89c922dc2e7e

2020-08-24 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96730 --- Comment #3 from Martin Jambor --- I have proposed a fix on the mailing list: https://gcc.gnu.org/pipermail/gcc-patches/2020-August/552488.html

[Bug tree-optimization/96730] [10/11 Regression] ICE on x86_64-linux-gnu with `-O1` to `-O3` (in verify_sra_access_forest, at tree-sra.c:2352) since r10-6320-g5b9e89c922dc2e7e

2020-08-24 Thread jamborm at gcc dot gnu.org
at gcc dot gnu.org |jamborm at gcc dot gnu.org --- Comment #2 from Martin Jambor --- Mine.

[Bug target/90234] 503.bwaves_r is 6% slower on Zen1/Zen2 CPUs at -Ofast with native march/mtune than with generic ones

2020-07-30 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90234 Martin Jambor changed: What|Removed |Added Summary|503.bwaves_r is 6% slower |503.bwaves_r is 6% slower

[Bug target/84490] [8/9/10/11 regression] 436.cactusADM regressed by 6-8% percent with -Ofast on Zen and Haswell, compared to gcc 7.2

2020-07-30 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84490 --- Comment #15 from Martin Jambor --- The problem sometimes is still there, sometimes it isn't: https://lnt.opensuse.org/db_default/v4/SPEC/graph?plot.0=37.100.0=27.100.0; I wonder whether we should keep this bug opened, the benchmark seems

[Bug target/84481] [8/9/10/11 Regression] 429.mcf with -O2 regresses by ~6% and ~4%, depending on tuning, on Zen compared to GCC 7.2

2020-07-30 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84481 --- Comment #12 from Martin Jambor --- I can once again confirm the slowdown on a zen1-based machine (commit 6e1e0decc9e vs gcc 7.5) but it is not present on a zen2-based one. I wonder whether the bug should me marked as WONTFIX.

[Bug ipa/93385] [10/11 Regression] wrong code with u128 modulo at -O2 -fno-dce -fno-ipa-cp -fno-tree-dce

2020-07-27 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93385 Martin Jambor changed: What|Removed |Added CC||suochenyao at 163 dot com --- Comment

[Bug ipa/96235] Segmentation fault with "-Og -fno-dce -fno-tree-dce -finline-small-functions -fipa-sra"

2020-07-27 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96235 Martin Jambor changed: What|Removed |Added Resolution|--- |DUPLICATE Status|NEW

[Bug ipa/96291] [10/11 Regression] -flto fails as "internal compiler error: Segmentation fault" during IPA pass: cp incall_for_symbol_thunks_and_aliases()

2020-07-27 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96291 Martin Jambor changed: What|Removed |Added Assignee|jamborm at gcc dot gnu.org |slyfox at inbox dot ru

[Bug ipa/96235] Segmentation fault with "-Og -fno-dce -fno-tree-dce -finline-small-functions -fipa-sra"

2020-07-23 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96235 --- Comment #6 from Martin Jambor --- (In reply to Martin Liška from comment #4) > It seems to me something related to IPA SRA. > @Martin: Can you please take a look? I will but -fno-dce -fno-tree-dce strongly suggest this is a duplicate of PR

[Bug ipa/96291] [10/11 Regression] -flto fails as "internal compiler error: Segmentation fault" during IPA pass: cp incall_for_symbol_thunks_and_aliases()

2020-07-23 Thread jamborm at gcc dot gnu.org
at gcc dot gnu.org |jamborm at gcc dot gnu.org --- Comment #2 from Martin Jambor --- I guess I should take a look

[Bug ipa/96040] [10/11 Regression] Compiled code causes SIGBUS at -O2

2020-07-04 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96040 Martin Jambor changed: What|Removed |Added Resolution|--- |FIXED Status|ASSIGNED

[Bug ipa/96040] [10/11 Regression] Compiled code causes SIGBUS at -O2

2020-07-03 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96040 --- Comment #9 from Martin Jambor --- True. Richi expressed preference for avoiding the transform when there are type mismatches, so I'm currently bootstrapping that. I guess we can always revisit the decision if we ever discover it would be

[Bug ipa/96040] [10/11 Regression] Compiled code causes SIGBUS at -O2

2020-07-03 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96040 --- Comment #7 from Martin Jambor --- Yes, IPA-SRA identifies accesses by both offset and size, so the situation would not have happened if the size was different.

[Bug ipa/96040] [10/11 Regression] Compiled code causes SIGBUS at -O2

2020-07-03 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96040 --- Comment #5 from Martin Jambor --- IPA-split puts the double access to the union in the .part function and keeps only the long int access in the "original" function. IPA-SRA thinks it can work with that but the code in "transitive" call

[Bug ipa/96040] [10/11 Regression] Compiled code causes SIGBUS at -O2

2020-07-03 Thread jamborm at gcc dot gnu.org
at gcc dot gnu.org |jamborm at gcc dot gnu.org --- Comment #4 from Martin Jambor --- I'll have a look

[Bug bootstrap/95970] gcc/go/gofrontend/types.cc:1474:34: warning: ‘this’ pointer null

2020-06-29 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95970 Martin Jambor changed: What|Removed |Added Last reconfirmed||2020-06-29 Ever confirmed|0

[Bug tree-optimization/95113] [10/11 Regression] Wrong code w/ -O2 -fexceptions -fnon-call-exceptions

2020-06-08 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95113 Martin Jambor changed: What|Removed |Added Resolution|--- |FIXED Status|NEW

[Bug tree-optimization/95113] [10/11 Regression] Wrong code w/ -O2 -fexceptions -fnon-call-exceptions

2020-06-08 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95113 --- Comment #7 from Martin Jambor --- Fixed. Thanks for reporting.

[Bug ipa/93385] [10/11 Regression] wrong code with u128 modulo at -O2 -fno-dce -fno-ipa-cp -fno-tree-dce

2020-06-08 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93385 Bug 93385 depends on bug 95113, which changed state. Bug 95113 Summary: [10/11 Regression] Wrong code w/ -O2 -fexceptions -fnon-call-exceptions https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95113 What|Removed

[Bug debug/95343] IPA-SRA can result in wrong debug info about removed function arguments

2020-05-28 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95343 --- Comment #3 from Martin Jambor --- I have proposed a patch series on the mailing list to address PR 93385 and the last patch in it also addresses this issue and allows gdb to print the correct value of the removed parameter:

[Bug tree-optimization/95113] [10/11 Regression] Wrong code w/ -O2 -fexceptions -fnon-call-exceptions

2020-05-28 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95113 --- Comment #4 from Martin Jambor --- (In reply to Arseny Solokha from comment #3) > > Indeed, -fno-ipa-sra fixes it. So, a duplicate of PR93385? Similar, but not quite the same. I have proposed a fix on the mailing list:

[Bug ipa/93385] [10/11 Regression] wrong code with u128 modulo at -O2 -fno-dce -fno-ipa-cp -fno-tree-dce

2020-05-28 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93385 --- Comment #35 from Martin Jambor --- I have proposed a patch series that deals with this issue, including proper adjustments to debug info, on the mailing list: https://gcc.gnu.org/pipermail/gcc-patches/2020-May/546702.html

[Bug web/95380] ipcp-unit-growth was renamed to ipa-cp-unit-growth

2020-05-28 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95380 --- Comment #4 from Martin Jambor --- (In reply to Martin Liška from comment #3) > Fixed for master, not planning to backport that. Why not? Are any of the parameters only in GCC 11? Should I prepare a special GCC 10 patch just to address the

[Bug debug/95343] IPA-SRA can result in wrong debug info about removed function arguments

2020-05-26 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95343 --- Comment #2 from Martin Jambor --- (In reply to Martin Jambor from comment #1) > ...I am testing a patch which can make gdb actually show > the correct 4. I meant the correct value 2, of course.

[Bug debug/95343] IPA-SRA can result in wrong debug info about removed function arguments

2020-05-26 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95343 Martin Jambor changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |jamborm at gcc dot gnu.org

[Bug debug/95343] New: IPA-SRA can result in bad debug info about removed function arguments

2020-05-26 Thread jamborm at gcc dot gnu.org
Priority: P3 Component: debug Assignee: unassigned at gcc dot gnu.org Reporter: jamborm at gcc dot gnu.org Target Milestone: --- Host: x86_64-linux Target: x86_64-linux Created attachment 48608 --> https://gcc.gnu.org/bugzi

[Bug target/95336] Bad code gen omnetpp_r aarch64

2020-05-26 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95336 --- Comment #6 from Martin Jambor --- (In reply to Erick Ochoa from comment #0) [...] > I did a bisection from > > commit f47f687a97260b1a1305cbf2d7ee3d74b2916a74 > Author: Richard Biener > Date: Thu Apr 25 17:58:56 2019 + > > to: >

[Bug libgomp/68033] OpenMP: ICE with teams distribute

2020-05-12 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68033 Martin Jambor changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug ipa/94856] [10/11 Regression] ICE: Segmentation fault (in clone_of_p); or ICE: verify_cgraph_node failed (error: edge points to wrong declaration) since r10-4944-g1e83bd7003e03160

2020-04-30 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94856 Martin Jambor changed: What|Removed |Added Resolution|--- |FIXED Status|NEW

[Bug ipa/94856] [10 Regression] ICE: Segmentation fault (in clone_of_p); or ICE: verify_cgraph_node failed (error: edge points to wrong declaration) since r10-4944-g1e83bd7003e03160

2020-04-30 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94856 --- Comment #8 from Martin Jambor --- I proposed the patch on the mailing list: https://gcc.gnu.org/pipermail/gcc-patches/2020-April/544943.html

[Bug ipa/94856] [10 Regression] ICE: Segmentation fault (in clone_of_p); or ICE: verify_cgraph_node failed (error: edge points to wrong declaration) since r10-4944-g1e83bd7003e03160

2020-04-30 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94856 --- Comment #7 from Martin Jambor --- The "edge points to wrong decl" case is a verifier error. We have a method which (in the course of IPA-CP) loses its this pointer because it is unused and the pass then does not clone all the this adjusting

[Bug ipa/94472] 400.perlbench is slower when compiled at -O2 with both PGO and LTO on AMD Zen CPUs

2020-04-28 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94472 --- Comment #3 from Martin Jambor --- My benchmarking setup is currently gone so unfortunately no, not easily. I'll be re-measuring everything on a different computer with a slightly different CPU model soon, so after that I guess I could. But

[Bug tree-optimization/94482] [8/9 Regression] Inserting into vector with optimization enabled on x86 generates incorrect result

2020-04-21 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94482 Martin Jambor changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug ipa/93385] [10 Regression] wrong code with u128 modulo at -O2 -fno-dce -fno-ipa-cp -fno-tree-dce

2020-04-20 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93385 --- Comment #30 from Martin Jambor --- Created attachment 48320 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=48320=edit Todays WIP patch This is my todays (still very much) WIP patch. - It marks statements which should not be copied

[Bug ipa/93385] [10 Regression] wrong code with u128 modulo at -O2 -fno-dce -fno-ipa-cp -fno-tree-dce

2020-04-17 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93385 --- Comment #25 from Martin Jambor --- (In reply to rguent...@suse.de from comment #21) > Btw, I'd much prefer to not first copy the stmts and then remove them. > Instead the DCE "analysis" can be done on the original IL and stmts > be "marked"

[Bug ipa/93385] [10 Regression] wrong code with u128 modulo at -O2 -fno-dce -fno-ipa-cp -fno-tree-dce

2020-04-17 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93385 --- Comment #22 from Martin Jambor --- (In reply to Jakub Jelinek from comment #18) > Comment on attachment 48302 [details] > Untested fix > > + /* IPA-SRA does not analyze other types of statements. */ > + gcc_unreachable ();

<    1   2   3   4   5   6   7   8   9   10   >