[Bug bootstrap/47402] [4.6 Regression] Bootstrap comparison failure
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47402 --- Comment #13 from Jan Hubicka hubicka at gcc dot gnu.org 2011-02-17 16:18:34 UTC --- Author: hubicka Date: Thu Feb 17 16:18:24 2011 New Revision: 170249 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=170249 Log: PR debug/47106 PR debug/47402 * cfgexpand.c (account_used_vars_for_block): Remove. (estimated_stack_frame_size): Use referenced vars. * tree-inline.c (remap_decl): Only mark VAR_DECLs as referenced that were referenced in the original function. Test src_fn rather than cfun. Drop redundant get_var_ann. (setup_one_parameter): Drop redundant get_var_ann. (declare_return_variable): Likewise. (copy_decl_for_dup_finish): Mark VAR_DECLs referenced in src_fn. (copy_arguments_for_versioning): Drop redundant get_var_ann. * ipa-inline.c (compute_inline_parameters): Do not compute disregard_inline_limits here. are not available. (compute_inlinable_for_current, pass_inlinable): New. (pass_inline_parameters): Require PROP_referenced_vars. * cgraphunit.c (cgraph_process_new_functions): Don't run compute_inline_parameters explicitly unless function is in SSA form. (cgraph_analyze_function): Set .disregard_inline_limits. * tree-sra.c (convert_callers): Compute inliner parameters only for functions already in SSA form. * g++.dg/debug/pr47106.C: New. Added: trunk/gcc/testsuite/g++.dg/debug/pr47106.C Modified: trunk/gcc/ChangeLog trunk/gcc/cfgexpand.c trunk/gcc/cgraphunit.c trunk/gcc/ipa-inline.c trunk/gcc/passes.c trunk/gcc/testsuite/ChangeLog trunk/gcc/tree-inline.c trunk/gcc/tree-sra.c
[Bug bootstrap/47402] [4.6 Regression] Bootstrap comparison failure
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47402 --- Comment #10 from Alexandre Oliva aoliva at gcc dot gnu.org 2011-02-15 18:36:35 UTC --- Author: aoliva Date: Tue Feb 15 18:36:31 2011 New Revision: 170186 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=170186 Log: PR debug/47106 PR debug/47402 * tree-flow.h (FOR_EACH_REFERENCED_VAR): Add FN argument. Adjust all users. Pass FN to... * tree-flow-inline.h (first_referenced_var): ... this. Add fn argument. * ipa-struct-reorg.c: Adjust. * tree-dfa.c: Adjust. * tree-into-ssa.c: Adjust. * tree-sra.c: Adjust. * tree-ssa-alias.c: Adjust. * tree-ssa-live.c: Adjust. * tree-ssa.c: Adjust. * tree-ssanames.c: Adjust. * tree-tailcall.c: Adjust. Modified: trunk/gcc/ChangeLog trunk/gcc/ipa-struct-reorg.c trunk/gcc/tree-dfa.c trunk/gcc/tree-flow-inline.h trunk/gcc/tree-flow.h trunk/gcc/tree-into-ssa.c trunk/gcc/tree-sra.c trunk/gcc/tree-ssa-alias.c trunk/gcc/tree-ssa-live.c trunk/gcc/tree-ssa.c trunk/gcc/tree-ssanames.c trunk/gcc/tree-tailcall.c
[Bug bootstrap/47402] [4.6 Regression] Bootstrap comparison failure
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47402 --- Comment #9 from Alexandre Oliva aoliva at gcc dot gnu.org 2011-02-15 18:36:21 UTC --- Author: aoliva Date: Tue Feb 15 18:36:16 2011 New Revision: 170185 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=170185 Log: PR debug/47106 PR debug/47402 * tree-flow.h (referenced_var_lookup): Add fn parameter. Adjust all callers. * tree-dfa.c (referenced_var_lookup): Use fn instead of cfun. * tree-flow-inline.h: Adjust. * gimple-pretty-print.c: Adjust. * tree-into-ssa.c: Adjust. * tree-ssa.c: Adjust. * cfgexpand.c: Adjust. Modified: trunk/gcc/ChangeLog trunk/gcc/cfgexpand.c trunk/gcc/gimple-pretty-print.c trunk/gcc/tree-dfa.c trunk/gcc/tree-flow-inline.h trunk/gcc/tree-flow.h trunk/gcc/tree-into-ssa.c trunk/gcc/tree-ssa.c
[Bug bootstrap/47402] [4.6 Regression] Bootstrap comparison failure
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47402 --- Comment #12 from Alexandre Oliva aoliva at gcc dot gnu.org 2011-02-15 18:37:01 UTC --- Author: aoliva Date: Tue Feb 15 18:36:56 2011 New Revision: 170188 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=170188 Log: PR debug/47106 PR debug/47402 * cgraph.h (compute_inline_parameters): Return void. * ipa-inline.c (compute_inline_parameters): Adjust. Modified: trunk/gcc/ChangeLog trunk/gcc/cgraph.h trunk/gcc/ipa-inline.c
[Bug bootstrap/47402] [4.6 Regression] Bootstrap comparison failure
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47402 --- Comment #11 from Alexandre Oliva aoliva at gcc dot gnu.org 2011-02-15 18:36:48 UTC --- Author: aoliva Date: Tue Feb 15 18:36:44 2011 New Revision: 170187 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=170187 Log: PR debug/47106 PR debug/47402 * tree-inline.h (estimated_stack_frame_size): Take cgraph node rather than decl. * cfgexpand.c (estimated_stack_frame_size): Likewise. * ipa-inline.c (compute_inline_parameters): Adjust. Modified: trunk/gcc/ChangeLog trunk/gcc/cfgexpand.c trunk/gcc/ipa-inline.c trunk/gcc/tree-inline.h
[Bug bootstrap/47402] [4.6 Regression] Bootstrap comparison failure
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47402 --- Comment #6 from Alexandre Oliva aoliva at gcc dot gnu.org 2011-02-02 05:44:44 UTC --- Author: aoliva Date: Wed Feb 2 05:44:38 2011 New Revision: 169514 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=169514 Log: PR debug/47106 PR debug/47402 * tree-inline.c (declare_return_variable): Add result decl to local decls only once. * gimple-low.c (record_vars_into): Mark newly-created variables as referenced. Modified: trunk/gcc/ChangeLog trunk/gcc/gimple-low.c trunk/gcc/tree-inline.c
[Bug bootstrap/47402] [4.6 Regression] Bootstrap comparison failure
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47402 --- Comment #7 from Alexandre Oliva aoliva at gcc dot gnu.org 2011-02-02 06:18:07 UTC --- Author: aoliva Date: Wed Feb 2 06:18:01 2011 New Revision: 169515 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=169515 Log: PR debug/47106 PR debug/47402 * tree-flow-inline.h (clear_is_used, is_used_p): New. * cfgexpand.c (account_used_vars_for_block): Use them. * tree-nrv.c (tree_nrv): Likewise. * tree-ssa-live.c (remove_unused_scope_block_p): Likewise. (dump_scope_block): Likewise. (remove_unused_locals): Likewise. Modified: trunk/gcc/ChangeLog trunk/gcc/cfgexpand.c trunk/gcc/tree-flow-inline.h trunk/gcc/tree-nrv.c trunk/gcc/tree-ssa-live.c
[Bug bootstrap/47402] [4.6 Regression] Bootstrap comparison failure
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47402 --- Comment #8 from Alexandre Oliva aoliva at gcc dot gnu.org 2011-02-02 06:28:26 UTC --- Author: aoliva Date: Wed Feb 2 06:28:22 2011 New Revision: 169516 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=169516 Log: PR debug/47106 PR debug/47402 * tree-inline.c (declare_return_variable): Remove unused caller variable. Modified: trunk/gcc/ChangeLog trunk/gcc/tree-inline.c
[Bug bootstrap/47402] [4.6 Regression] Bootstrap comparison failure
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47402 Dominique d'Humieres dominiq at lps dot ens.fr changed: What|Removed |Added CC||aoliva at gcc dot gnu.org --- Comment #1 from Dominique d'Humieres dominiq at lps dot ens.fr 2011-01-21 18:46:41 UTC --- According http://gcc.gnu.org/ml/gcc-regression/2011-01/msg00345.html this is due to revision 169093.
[Bug bootstrap/47402] [4.6 Regression] Bootstrap comparison failure
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47402 H.J. Lu hjl.tools at gmail dot com changed: What|Removed |Added Priority|P3 |P1 Status|UNCONFIRMED |NEW Last reconfirmed||2011.01.21 18:54:25 Target Milestone|--- |4.6.0 Ever Confirmed|0 |1 --- Comment #2 from H.J. Lu hjl.tools at gmail dot com 2011-01-21 18:54:25 UTC --- Revision 169093: http://gcc.gnu.org/ml/gcc-cvs/2011-01/msg00715.html is the cause.
[Bug bootstrap/47402] [4.6 Regression] Bootstrap comparison failure
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47402 --- Comment #3 from Alexandre Oliva aoliva at gcc dot gnu.org 2011-01-21 22:11:28 UTC --- Author: aoliva Date: Fri Jan 21 22:11:24 2011 New Revision: 169107 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=169107 Log: PR debug/47402 Temporarily revert: 2011-01-21 Alexandre Oliva aol...@redhat.com PR debug/47106 * tree-dfa.c (create_var_ann): Mark variable as used. Modified: trunk/gcc/ChangeLog trunk/gcc/tree-dfa.c
[Bug bootstrap/47402] [4.6 Regression] Bootstrap comparison failure
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47402 --- Comment #4 from Alexandre Oliva aoliva at gcc dot gnu.org 2011-01-21 22:35:47 UTC --- Ok, regression now fixed by reversal of the bit that exposed the failure, but the fact that it regressed implies there's something iffy with the patch for bug 47106, so I'll keep this open while I try to figure that out. I'll unfortunately be away tomorrow (back on Sunday), so if the patch for 47106 causes further breakage, please revert the rest of it. Sorry, and thanks in advance,
[Bug bootstrap/47402] [4.6 Regression] Bootstrap comparison failure
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47402 H.J. Lu hjl.tools at gmail dot com changed: What|Removed |Added Status|NEW |RESOLVED Resolution||FIXED --- Comment #5 from H.J. Lu hjl.tools at gmail dot com 2011-01-22 01:10:45 UTC --- Fixed.