[Bug tree-optimization/93776] [10 Regression] ICE in verify_sra_access_forest, at tree-sra.c:2326

2020-02-17 Thread jamborm at gcc dot gnu.org
at gcc dot gnu.org |jamborm at gcc dot gnu.org --- Comment #2 from Martin Jambor --- But it is actually a slightly different problem. The assignment to d7.pr, which is an empty struct, survives all the way to gimple as a load from an initializer of size zero, which makes SRA to create an

[Bug tree-optimization/93516] [10 regression] ICE in verify_sra_access_forest, at tree-sra.c:2342 since r10-6322

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

[Bug tree-optimization/93516] [10 regression] ICE in verify_sra_access_forest, at tree-sra.c:2342 since r10-6322

2020-02-14 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93516 --- Comment #6 from Martin Jambor --- I proposed a fix on the mailing list: https://gcc.gnu.org/ml/gcc-patches/2020-02/msg00850.html

[Bug c++/93667] [10 regression] ICE in esra with nested [[no_unique_address]] field

2020-02-13 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93667 --- Comment #5 from Martin Jambor --- It is easy to prevent the ICE with the following, which prevents total scalarization from happening. However, if someone marked a field with such an attribute, the encompassing structure perhaps should be op

[Bug c++/93667] [10 regression] ICE in esra with nested [[no_unique_address]] field

2020-02-11 Thread jamborm at gcc dot gnu.org
at gcc dot gnu.org |jamborm at gcc dot gnu.org --- Comment #4 from Martin Jambor --- Mine.

[Bug tree-optimization/93516] [10 regression] ICE in verify_sra_access_forest, at tree-sra.c:2342 since r10-6322

2020-02-07 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93516 --- Comment #5 from Martin Jambor --- So the problems seems to be that get_ref_base_an_extent returns different sizes for expressions of the same type - specifically a RECORD_TYPE with padding - when one is a standalone VAR_DECL and the other a

[Bug ipa/92548] FAIL: gcc.dg/ipa/ipa-sra-12.c scan-ipa-dump-times sra "Will split parameter" 2

2020-02-05 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92548 --- Comment #2 from Martin Jambor --- The problem is that IPA-SRA does not handle structures passed by invisible reference and 32bit hppa uses it. I would suggest xfailing the scan it like we did in PR77732. John, since you can actually run the

[Bug tree-optimization/93516] [10 regression] ICE in verify_sra_access_forest, at tree-sra.c:2342 since r10-6322

2020-01-31 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93516 --- Comment #3 from Martin Jambor --- I'll have a look.

[Bug c++/92700] wrong "unintialized" warning with std::optional

2020-01-30 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92700 --- Comment #4 from Martin Jambor --- At first glance this looks like a dup of PR80635

[Bug bootstrap/93214] [10 Regression] Ada LTO bootstrap fails with undefined reference to __gnat_debug_raise_assert_failure

2020-01-29 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93214 --- Comment #6 from Martin Jambor --- Indeed, the first fixed revision was r10-6023-f48c6014133c8989702458f9082e34ba6dd326d4: f48c6014133c8989702458f9082e34ba6dd326d4 is the first fixed commit commit f48c6014133c8989702458f9082e34ba6dd326d4 Auth

[Bug bootstrap/93214] [10 Regression] Ada LTO bootstrap fails with undefined reference to __gnat_debug_raise_assert_failure

2020-01-27 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93214 --- Comment #3 from Martin Jambor --- I don't know what helped but I was able to LTO bootstrap Ada today, specifically master commit r10-6244-e97a3063fb1 (on x86_64-linux).

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

2020-01-23 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93385 --- Comment #14 from Martin Jambor --- Another option, which does not create an inter-pass dependency and does not clutter tree-inline any more, but which pessimizes IPA-SRA (put perhaps just alittle bit?), is making sure that the statements whic

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

2020-01-22 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93385 --- Comment #13 from Martin Jambor --- The assumption is that DCE will eventually remove all gimple assignments with LHS which has no uses if gimple_has_volatile_ops returns false for them. I have had only a brief look at DCE today and it seems

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

2020-01-22 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93385 --- Comment #11 from Martin Jambor --- I'm going to test a patch that basically does the following (plus moving push_cfun a bit): diff --git a/gcc/ipa-sra.c b/gcc/ipa-sra.c index 31de527d111..e18dc6958dc 100644 --- a/gcc/ipa-sra.c +++ b/gcc/ipa-

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

2020-01-22 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93385 --- Comment #10 from Martin Jambor --- Ah, -fno-tree-dce is specfied on the command line...

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

2020-01-22 Thread jamborm at gcc dot gnu.org
at gcc dot gnu.org |jamborm at gcc dot gnu.org --- Comment #8 from Martin Jambor --- Yes, IPA-SRA removes the parameter because we do bunch of computations on them but never store the result anywhere nor pass it to another function. The problem is that the modulo operation survives the

[Bug tree-optimization/93271] [8/9/10 regression] SRA producing wrong code on denormals

2020-01-15 Thread jamborm at gcc dot gnu.org
||2020-01-15 Assignee|unassigned at gcc dot gnu.org |jamborm at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #7 from Martin Jambor --- I'll take a look

[Bug ipa/92240] [10 regression] ICE in duplicate, at ipa-prop.c:3883

2020-01-14 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92240 Martin Jambor changed: What|Removed |Added CC||jamborm at gcc dot gnu.org --- Comment

[Bug ipa/93223] [9/10 Regression] ICE in devirtualization_time_bonus at gcc/ipa-cp.c:3161 since r261744

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

[Bug ipa/92971] Suspicious code in cgraph_edge_brings_all_agg_vals_for_node(), ipa-cp.c

2020-01-10 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92971 --- Comment #7 from Martin Jambor --- Author: jamborm Date: Fri Jan 10 22:46:46 2020 New Revision: 280151 URL: https://gcc.gnu.org/viewcvs?rev=280151&root=gcc&view=rev Log: IPA-CP: Remove bogus static keyword (PR 92971) 2020-01-10 Martin Jambo

[Bug ipa/92971] Suspicious code in cgraph_edge_brings_all_agg_vals_for_node(), ipa-cp.c

2020-01-10 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92971 --- Comment #6 from Martin Jambor --- Author: jamborm Date: Fri Jan 10 19:30:56 2020 New Revision: 280131 URL: https://gcc.gnu.org/viewcvs?rev=280131&root=gcc&view=rev Log: IPA-CP: Remove bogus static keyword (PR 92971) 2020-01-10 Martin Jambo

[Bug ipa/93223] [9/10 Regression] ICE in devirtualization_time_bonus at gcc/ipa-cp.c:3161 since r261744

2020-01-10 Thread jamborm at gcc dot gnu.org
at gcc dot gnu.org |jamborm at gcc dot gnu.org --- Comment #2 from Martin Jambor --- (In reply to Christoph Höger from comment #0) > it looks like the result of ipa_fn_summaries->get (callee); should be > checked for nullptr. Indeed, we figure out that after cloning we could devirt

[Bug lto/93166] [10 Regression] ICE in get_info_about_necessary_edges, at ipa-cp.c:4137 since r278893

2020-01-10 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93166 --- Comment #2 from Martin Jambor --- I have analyzed this ICE and came to the conclusion that the assert is wrong for polymorphic context lattices - e.g. in the reported case we always pass the same class to first parameter, which in the recursi

[Bug bootstrap/56593] LTO profiledbootstrap fails for Ada

2020-01-09 Thread jamborm at gcc dot gnu.org
||jamborm at gcc dot gnu.org Resolution|--- |FIXED --- Comment #10 from Martin Jambor --- At least r279561 can be bootstrapped on an x86_64-linux (the subsequent r279563 breaks normal Ada LTO bootstrapped, see PR 93214) so I believe this old bug

[Bug bootstrap/93214] New: Ada LTO bootstrap fails with undefined reference to __gnat_debug_raise_assert_failure

2020-01-09 Thread jamborm at gcc dot gnu.org
Severity: 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

[Bug ipa/92917] [10 Regression] ICE in print_all_lattices at gcc/ipa-cp.c:547 since r278016.

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

[Bug ipa/92917] [10 Regression] ICE in print_all_lattices at gcc/ipa-cp.c:547 since r278016.

2020-01-03 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92917 --- Comment #2 from Martin Jambor --- Author: jamborm Date: Fri Jan 3 13:52:38 2020 New Revision: 279859 URL: https://gcc.gnu.org/viewcvs?rev=279859&root=gcc&view=rev Log: Avoid segfault when dumping IPA-CP lattices for unoptimized functions (P

[Bug ipa/93015] [10 Regression] Segmentation fault (ipcp_store_vr_results(void))

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

[Bug tree-optimization/91579] tailr1 pass creates redundant phi nodes

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

[Bug tree-optimization/93084] [10 regression] Infinite loop in ipa-cp when building clang with LTO+PGO

2020-01-02 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93084 --- Comment #10 from Martin Jambor --- (In reply to fxue from comment #8) > [...] > Then is there a case that a ipcp_lattice be shared by different cgraph nodes? No, there isn't.

[Bug tree-optimization/93084] [10 regression] Infinite loop in ipa-cp when building clang with LTO+PGO

2019-12-28 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93084 Martin Jambor changed: What|Removed |Added CC||jamborm at gcc dot gnu.org --- Comment

[Bug ipa/93015] [10 Regression] Segmentation fault (ipcp_store_vr_results(void))

2019-12-21 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93015 --- Comment #4 from Martin Jambor --- Author: jamborm Date: Sat Dec 21 11:25:05 2019 New Revision: 279695 URL: https://gcc.gnu.org/viewcvs?rev=279695&root=gcc&view=rev Log: Avoid segfault when doing IPA-VRP but not IPA-CP (PR 93015) 2019-12-21

[Bug ipa/93015] [10 Regression] Segmentation fault (ipcp_store_vr_results(void))

2019-12-20 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93015 --- Comment #3 from Martin Jambor --- I proposed a patch on the mailing list: https://gcc.gnu.org/ml/gcc-patches/2019-12/msg01451.html

[Bug ipa/93015] [10 Regression] Segmentation fault (ipcp_store_vr_results(void))

2019-12-20 Thread jamborm at gcc dot gnu.org
at gcc dot gnu.org |jamborm at gcc dot gnu.org --- Comment #2 from Martin Jambor --- Heh, ipcp_store_vr_results checks for flag_ipa_vrp but not for flag_ipa_cp or optimize, which means that it accesses info which has not been created. I suppose it's best to do what ipcp_store_bits_re

[Bug ipa/92971] Suspicious code in cgraph_edge_brings_all_agg_vals_for_node(), ipa-cp.c

2019-12-18 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92971 Martin Jambor changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug ipa/92971] Suspicious code in cgraph_edge_brings_all_agg_vals_for_node(), ipa-cp.c

2019-12-18 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92971 --- Comment #4 from Martin Jambor --- Author: jamborm Date: Wed Dec 18 16:08:09 2019 New Revision: 279525 URL: https://gcc.gnu.org/viewcvs?rev=279525&root=gcc&view=rev Log: IPA-CP: Remove bogus static keyword (PR 92971) 2019-12-18 Martin Jambo

[Bug tree-optimization/92486] Wrong optimization: padding in structs is not copied even with memcpy

2019-12-17 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92486 --- Comment #17 from Martin Jambor --- If we really decide to fix this in SRA, i can be done (after the previous patches in the series are in) with something like https://gcc.gnu.org/ml/gcc-patches/2019-12/msg01185.html

[Bug tree-optimization/92706] SRA confuses FRE

2019-12-17 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92706 --- Comment #4 from Martin Jambor --- I have proposed the following patches to address this on trunk. The testcase from comment #3 can be fixed with https://gcc.gnu.org/ml/gcc-patches/2019-12/msg01183.html The original testcase however needs al

[Bug ipa/92971] Suspicious code in cgraph_edge_brings_all_agg_vals_for_node(), ipa-cp.c

2019-12-17 Thread jamborm at gcc dot gnu.org
||2019-12-17 Assignee|unassigned at gcc dot gnu.org |jamborm at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #3 from Martin Jambor --- I'll remove it with https://gcc.gnu.org/ml/gcc-patches/2019-12/msg01186.html

[Bug ipa/92971] Suspicious code in cgraph_edge_brings_all_agg_vals_for_node(), ipa-cp.c

2019-12-17 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92971 --- Comment #2 from Martin Jambor --- (In reply to fxue from comment #0) > The variable "values" is defined as static, which makes a questionable side > effect. History calls will impact result of current call! > > for (i = 0; i < count; i++)

[Bug ipa/92676] [10 Regression] lto1: error: comdat-local function called by construct.constprop outside its comdat since r278669

2019-12-16 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92676 --- Comment #3 from Martin Jambor --- I proposed a fix on the mailing list: https://gcc.gnu.org/ml/gcc-patches/2019-12/msg01129.html

[Bug tree-optimization/92706] SRA confuses FRE

2019-12-03 Thread jamborm at gcc dot gnu.org
at gcc dot gnu.org |jamborm at gcc dot gnu.org

[Bug c++/68230] Unused function parameters not reported by -Wunused-parameter when only used recursively (add -Wparameter-only-used-recursively instead?)

2019-12-02 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68230 --- Comment #12 from Martin Jambor --- (In reply to Eric Gallager from comment #10) > Martin Jambor's IPA-SRA rewrite might be relevant here; it sounded like the > new IPA-SRA will remove parameters that are unused like this, but I didn't > quite

[Bug tree-optimization/92741] [10 Regression] ICE: verify_gimple failed (error: invalid vector types in nop conversion)

2019-12-02 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92741 --- Comment #5 from Martin Jambor --- (In reply to rsand...@gcc.gnu.org from comment #4) > This comes from: > > if (!useless_type_conversion_p (TREE_TYPE (rhs), TREE_TYPE (v->value))) > { > if (fold_convertible_p (TREE_TY

[Bug ipa/92133] Support multi versioning on self recursive function

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

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

2019-12-02 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=26163 Bug 26163 depends on bug 92133, which changed state. Bug 92133 Summary: Support multi versioning on self recursive function https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92133 What|Removed |Added

[Bug ipa/92697] IPA-SRA modifies ifunc_resolvers

2019-11-29 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92697 Martin Jambor changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug ipa/92476] [10 regression] SEGV in cgraph_edge_brings_value_p

2019-11-29 Thread jamborm at gcc dot gnu.org
||jamborm at gcc dot gnu.org Resolution|--- |FIXED Assignee|mjambor at suse dot cz |jamborm at gcc dot gnu.org --- Comment #7 from Martin Jambor --- Fixed.

[Bug ipa/92476] [10 regression] SEGV in cgraph_edge_brings_value_p

2019-11-29 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92476 --- Comment #6 from Martin Jambor --- Author: jamborm Date: Fri Nov 29 13:36:47 2019 New Revision: 278842 URL: https://gcc.gnu.org/viewcvs?rev=278842&root=gcc&view=rev Log: Add an x86_64 test for PR 92476 2019-11-29 Martin Jambor PR

[Bug ipa/92476] [10 regression] SEGV in cgraph_edge_brings_value_p

2019-11-29 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92476 --- Comment #5 from Martin Jambor --- Author: jamborm Date: Fri Nov 29 13:29:35 2019 New Revision: 278841 URL: https://gcc.gnu.org/viewcvs?rev=278841&root=gcc&view=rev Log: ipa-cp: Avoid ICEs when looking at expanded thunks and unoptimized funct

[Bug ipa/92697] IPA-SRA modifies ifunc_resolvers

2019-11-28 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92697 --- Comment #2 from Martin Jambor --- Author: jamborm Date: Thu Nov 28 15:39:48 2019 New Revision: 278812 URL: https://gcc.gnu.org/viewcvs?rev=278812&root=gcc&view=rev Log: cgraph: ifunc resolvers cannot be made local (PR 92697) 2019-11-28 Mar

[Bug ipa/92697] IPA-SRA modifies ifunc_resolvers

2019-11-27 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92697 --- Comment #1 from Martin Jambor --- And for the record, I'm testing a patch.

[Bug ipa/92697] New: IPA-SRA modifies ifunc_resolvers

2019-11-27 Thread jamborm at gcc dot gnu.org
Assignee: jamborm at gcc dot gnu.org Reporter: jamborm at gcc dot gnu.org CC: marxin at gcc dot gnu.org Target Milestone: --- Created attachment 47381 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=47381&action=edit testcase In the attached testcase, IPA-SRA

[Bug ipa/92109] [10 Regression] ICE in modify_call_stmt, at ipa-param-manipulation.c:1586

2019-11-25 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92109 Martin Jambor changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug ipa/91956] [10 Regression] ICE: verify_cgraph_node failed (error: comdat-local function called by __ct .isra outside its comdat) since r275982

2019-11-25 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91956 Martin Jambor changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug ipa/92109] [10 Regression] ICE in modify_call_stmt, at ipa-param-manipulation.c:1586

2019-11-25 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92109 --- Comment #8 from Martin Jambor --- Author: jamborm Date: Mon Nov 25 10:13:08 2019 New Revision: 278670 URL: https://gcc.gnu.org/viewcvs?rev=278670&root=gcc&view=rev Log: ipa: Prevent materialization of clones with removed bodies (PR 92109) 2

[Bug ipa/91956] [10 Regression] ICE: verify_cgraph_node failed (error: comdat-local function called by __ct .isra outside its comdat) since r275982

2019-11-25 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91956 --- Comment #3 from Martin Jambor --- Author: jamborm Date: Mon Nov 25 10:09:42 2019 New Revision: 278669 URL: https://gcc.gnu.org/viewcvs?rev=278669&root=gcc&view=rev Log: Make IPA-SRA follow comdat-local rules (PR 91956) 2019-11-25 Martin Ja

[Bug ipa/91832] [10 Regression] ICE in scan_expr_access, at ipa-sra.c:1695 since r275982

2019-11-23 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91832 Martin Jambor changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug ipa/92109] [10 Regression] ICE in modify_call_stmt, at ipa-param-manipulation.c:1586

2019-11-22 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92109 --- Comment #7 from Martin Jambor --- After talking to Honza, I have proposed a different patch on the mailing list: https://gcc.gnu.org/ml/gcc-patches/2019-11/msg02205.html

[Bug ipa/91956] [10 Regression] ICE: verify_cgraph_node failed (error: comdat-local function called by __ct .isra outside its comdat) since r275982

2019-11-22 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91956 --- Comment #2 from Martin Jambor --- I proposed a patch on the mailing list: https://gcc.gnu.org/ml/gcc-patches/2019-11/msg02203.html

[Bug ipa/92109] [10 Regression] ICE in modify_call_stmt, at ipa-param-manipulation.c:1586

2019-11-20 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92109 --- Comment #6 from Martin Jambor --- So this helps: diff --git a/gcc/cgraphclones.c b/gcc/cgraphclones.c index bfcebb20495..359ea53d8a6 100644 --- a/gcc/cgraphclones.c +++ b/gcc/cgraphclones.c @@ -1079,6 +1079,7 @@ symbol_table::materialize_all

[Bug ipa/92109] [10 Regression] ICE in modify_call_stmt, at ipa-param-manipulation.c:1586

2019-11-20 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92109 Martin Jambor changed: What|Removed |Added CC||hubicka at gcc dot gnu.org --- Comment #

[Bug ipa/92528] [10 Regression] ICE in ipa_get_parm_lattices since r278219

2019-11-18 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92528 --- Comment #11 from Martin Jambor --- Author: jamborm Date: Mon Nov 18 15:50:06 2019 New Revision: 278415 URL: https://gcc.gnu.org/viewcvs?rev=278415&root=gcc&view=rev Log: Add testcase for already fixed PR ipa/92528 2019-11-18 Martin Jambor

[Bug c++/80635] std::optional and bogus -Wmaybe-uninitialized warning

2019-11-18 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80635 --- Comment #28 from Martin Jambor --- The RFC did not receive any real negative feedback so I proposed to commit an updated patch: https://gcc.gnu.org/ml/gcc-patches/2019-11/msg01494.html

[Bug ipa/92528] [10 Regression] ICE in ipa_get_parm_lattices since r278219

2019-11-15 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92528 --- Comment #8 from Martin Jambor --- (In reply to Jan Hubicka from comment #6) > There is no need to clear polymorphic call context. It does not refer to the > parameters of caller. If it was valid for all possible contexts it is still > valid.

[Bug tree-optimization/92486] Wrong optimization: padding in structs is not copied even with memcpy

2019-11-15 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92486 --- Comment #15 from Martin Jambor --- (In reply to rguent...@suse.de from comment #14) > On Fri, 15 Nov 2019, jamborm at gcc dot gnu.org wrote: > > [...] But yes, for your example we'd copy-prop out c and b which > might

[Bug tree-optimization/92486] Wrong optimization: padding in structs is not copied even with memcpy

2019-11-15 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92486 --- Comment #13 from Martin Jambor --- (In reply to rguent...@suse.de from comment #10) > [...] But total scalarization works with the premise > that we don't see any direct accesses to source or destination That is not true, total scalarization

[Bug tree-optimization/92486] Wrong optimization: padding in structs is not copied even with memcpy

2019-11-14 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92486 --- Comment #5 from Martin Jambor --- While I don't share this preference about assignments, SRA only ignores padding when doing "total scalarization" aka the poor man's aggregate copy propagation because in that mode it would have to invent smal

[Bug lto/70929] [8/9 regression] Cross-module inlining for functions having argument passed by reference is no longer working.

2019-11-14 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70929 Martin Jambor changed: What|Removed |Added Summary|[8/9/10 regression] |[8/9 regression] |Cros

[Bug ipa/92497] New: Aggregate IPA-CP and inlining do not play well together, transformation is lost

2019-11-13 Thread jamborm at gcc dot gnu.org
: normal Priority: P3 Component: ipa Assignee: unassigned at gcc dot gnu.org Reporter: jamborm at gcc dot gnu.org CC: hubicka at gcc dot gnu.org, marxin at gcc dot gnu.org Target Milestone: --- The following breaks when compiled with -O2 -fno

[Bug ipa/92454] [10 Regression] ICE: Segmentation fault (in identify_dead_nodes)

2019-11-13 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92454 Martin Jambor changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug ipa/92454] [10 Regression] ICE: Segmentation fault (in identify_dead_nodes)

2019-11-13 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92454 --- Comment #6 from Martin Jambor --- Author: jamborm Date: Wed Nov 13 14:12:58 2019 New Revision: 278142 URL: https://gcc.gnu.org/viewcvs?rev=278142&root=gcc&view=rev Log: Add a few missing checks that IPA_NODE_REF is not NULL (PR 92454) 2019-

[Bug ipa/92454] [10 Regression] ICE: Segmentation fault (in identify_dead_nodes)

2019-11-12 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92454 Martin Jambor changed: What|Removed |Added Assignee|hubicka at gcc dot gnu.org |jamborm at gcc dot gnu.org

[Bug ipa/92454] [10 Regression] ICE: Segmentation fault (in identify_dead_nodes)

2019-11-11 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92454 --- Comment #3 from Martin Jambor --- I think the patch is (In reply to Jan Hubicka from comment #2) > This is the usual problem of trying to process node with no summary > attached to it. The following fixes the ICE, but I am not sure if there >

[Bug c++/80635] std::optional and bogus -Wmaybe-uninitialized warning

2019-11-10 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80635 --- Comment #27 from Martin Jambor --- (In reply to Manuel López-Ibáñez from comment #26) > Hi Martin, > > Wouldn't it be better if the testcase tested that no warning is given for a > true case? Otherwise if the bug is fixed, no warning will be

[Bug c++/80635] std::optional and bogus -Wmaybe-uninitialized warning

2019-11-08 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80635 --- Comment #25 from Martin Jambor --- I have posted an RFC patch alleviating the situation somewhat to the mailing list: https://gcc.gnu.org/ml/gcc-patches/2019-11/msg00614.html

[Bug ipa/92409] [10 regression] r277920 causes ICE in gcc.dg/cast-function-1.c

2019-11-08 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92409 --- Comment #11 from Martin Jambor --- (In reply to Richard Biener from comment #8) > @@ -3634,6 +3636,10 @@ declare_return_variable (copy_body_data > && !DECL_GIMPLE_REG_P (result) > && DECL_P (var)) > DECL_GIMPLE_REG

[Bug ipa/92409] [10 regression] r277920 causes ICE in gcc.dg/cast-function-1.c

2019-11-08 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92409 Martin Jambor changed: What|Removed |Added Attachment #47195|0 |1 is obsolete|

[Bug ipa/92409] [10 regression] r277920 causes ICE in gcc.dg/cast-function-1.c

2019-11-08 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92409 --- Comment #7 from Martin Jambor --- Ah sorry, looking at my bash history, yesterday I somehow lost the -m32 on my command line :-( Will try again.

[Bug ipa/92409] [10 regression] r277920 causes ICE in gcc.dg/cast-function-1.c

2019-11-07 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92409 --- Comment #3 from Martin Jambor --- Created attachment 47195 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=47195&action=edit Hopefully the fix I have to leave the office now but I am testing the attached fix on an x86_64 - I have lost c

[Bug other/92409] [10 regression] r277920 causes ICE in gcc.dg/cast-function-1.c

2019-11-07 Thread jamborm at gcc dot gnu.org
at gcc dot gnu.org |jamborm at gcc dot gnu.org --- Comment #2 from Martin Jambor --- Confirmed, at least on the i686, and mine.

[Bug lto/70929] [7/8/9/10 regression] Cross-module inlining for functions having argument passed by reference is no longer working.

2019-11-07 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70929 --- Comment #15 from Martin Jambor --- Author: jamborm Date: Thu Nov 7 10:55:43 2019 New Revision: 277920 URL: https://gcc.gnu.org/viewcvs?rev=277920&root=gcc&view=rev Log: Remove gimple_call_types_likely_match_p (PR 70929) 2019-11-07 Martin

[Bug lto/70929] [7/8/9/10 regression] Cross-module inlining for functions having argument passed by reference is no longer working.

2019-11-01 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70929 --- Comment #14 from Martin Jambor --- I sent a more detailed analysis of the behavior of gimple_call_types_likely_match_p and the idea from comment #5 along with a patch that mostly removes the predicate to the mailing list: https://gcc.gnu.org

[Bug ipa/92254] [10 regression] ICE LTO in inline_small_functions, at ipa-inline.c:2000

2019-10-30 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92254 Martin Jambor changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug ipa/92278] [10 regression] LTO ICE ipa_get_ith_polymorhic_call_context ipa-prop.h:616

2019-10-30 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92278 --- Comment #7 from Martin Jambor --- *** Bug 92254 has been marked as a duplicate of this bug. ***

[Bug ipa/92278] [10 regression] LTO ICE ipa_get_ith_polymorhic_call_context ipa-prop.h:616

2019-10-30 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92278 Martin Jambor changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug ipa/92278] [10 regression] LTO ICE ipa_get_ith_polymorhic_call_context ipa-prop.h:616

2019-10-30 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92278 --- Comment #5 from Martin Jambor --- See https://gcc.gnu.org/ml/gcc-patches/2019-10/msg02139.html for a possible fix.

[Bug ipa/92278] [10 regression] LTO ICE ipa_get_ith_polymorhic_call_context ipa-prop.h:616

2019-10-30 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92278 --- Comment #4 from Martin Jambor --- (In reply to Jan Hubicka from comment #3) > Martin, do you have any idea? Yes, the jump functions are thrown away at stream-in time because e->possibly_call_in_translation_unit_p returns false in: static v

[Bug middle-end/92037] [10 Regression] LTO bootstrap broken in selftests

2019-10-10 Thread jamborm at gcc dot gnu.org
||2019-10-10 CC||dmalcolm at gcc dot gnu.org, ||hubicka at gcc dot gnu.org, ||jamborm at gcc dot gnu.org Ever confirmed|0

[Bug ipa/60243] IPA is slow on large cgraph tree

2019-10-07 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60243 --- Comment #26 from Martin Jambor --- With new IPA-SRA, the situation has improved quite a bit, see below where old-ipa-sra is trunk r275981 and new-ipa-sra is trunk r275982 (arrival of new IPA-SRA): $ /usr/bin/time -f 'real=%e user=%U' taskset

[Bug preprocessor/91991] New: ICE in linemap_macro_map_lookup when LTO-building SQLite after r275402

2019-10-04 Thread jamborm at gcc dot gnu.org
Priority: P3 Component: preprocessor Assignee: unassigned at gcc dot gnu.org Reporter: jamborm at gcc dot gnu.org CC: hubicka at gcc dot gnu.org, nathan at gcc dot gnu.org Target Milestone: --- Host: x86_64-linux

[Bug gcov-profile/91969] New: Compiling testsuite/g++.dg/ipa/pr85421.C with -fdump-ipa-inline ICEs

2019-10-02 Thread jamborm at gcc dot gnu.org
Priority: P3 Component: gcov-profile Assignee: unassigned at gcc dot gnu.org Reporter: jamborm at gcc dot gnu.org CC: marxin at gcc dot gnu.org Target Milestone: --- Host: x86_64-linux Target: x86_64-linux If you

[Bug testsuite/91842] new test case gcc.dg/ipa/ipa-sra-19.c in r275982 has compilation error

2019-10-02 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91842 Martin Jambor changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug testsuite/91842] new test case gcc.dg/ipa/ipa-sra-19.c in r275982 has compilation error

2019-10-02 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91842 --- Comment #3 from Martin Jambor --- Author: jamborm Date: Wed Oct 2 15:09:37 2019 New Revision: 276465 URL: https://gcc.gnu.org/viewcvs?rev=276465&root=gcc&view=rev Log: [PR testsuite/91842] Skip gcc.dg/ipa/ipa-sra-19.c on power 2019-10-02

[Bug ipa/91853] [10 Regression] ICE in gimplify_modify_expr, at gimplify.c:5902 since r275982

2019-10-01 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91853 Martin Jambor changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug ipa/91853] [10 Regression] ICE in gimplify_modify_expr, at gimplify.c:5902 since r275982

2019-09-30 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91853 --- Comment #5 from Martin Jambor --- Author: jamborm Date: Mon Sep 30 08:18:59 2019 New Revision: 276296 URL: https://gcc.gnu.org/viewcvs?rev=276296&root=gcc&view=rev Log: [PR 91853] Prevent IPA-SRA ICEs on type-mismatched calls 2019-09-30 Ma

[Bug lto/70929] [7/8/9/10 regression] Cross-module inlining for functions having argument passed by reference is no longer working.

2019-09-27 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70929 Martin Jambor changed: What|Removed |Added CC||jamborm at gcc dot gnu.org --- Comment

[Bug ipa/91894] [10 Regression] ICE in tree check: expected tree that contains ‘decl common’ structure, have ‘integer_cst’ in tree_could_trap_p, at tree-eh.c:2679 since r275982

2019-09-25 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91894 Martin Jambor changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug ipa/91853] [10 Regression] ICE in gimplify_modify_expr, at gimplify.c:5902 since r275982

2019-09-25 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91853 --- Comment #4 from Martin Jambor --- *** Bug 91894 has been marked as a duplicate of this bug. ***

[Bug ipa/91872] [10 Regression] ICE in insert_vi_for_tree, at tree-ssa-structalias.c:2877, or ICE in expand_expr_real_1, at expr.c:10062

2019-09-24 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91872 Martin Jambor changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

<    3   4   5   6   7   8   9   10   11   12   >