[Bug target/45250] [4.6 Regression] FAIL: tr1/5_numerical_facilities/special_functions/01_assoc_laguerre/check_nan.cc
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45250 Jakub Jelinek changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution||FIXED --- Comment #14 from Jakub Jelinek 2010-11-01 08:00:00 UTC --- Fixed.
[Bug target/45250] [4.6 Regression] FAIL: tr1/5_numerical_facilities/special_functions/01_assoc_laguerre/check_nan.cc
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45250 --- Comment #13 from Jakub Jelinek 2010-11-01 07:58:51 UTC --- Author: jakub Date: Mon Nov 1 07:58:46 2010 New Revision: 166126 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=166126 Log: PR target/45250 * var-tracking.c (cfa_base_rtx): Move definition earlier in the file. (cfa_base_offset): New variable. (compute_cfa_pointer): Use cfa_base_rtx and cfa_base_offset. (adjust_mems): Don't do any sp or hfp replacements if cfa_base_rtx is NULL. (vt_init_cfa_base): Initialize cfa_base_offset. Modified: trunk/gcc/ChangeLog trunk/gcc/var-tracking.c
[Bug target/45250] [4.6 Regression] FAIL: tr1/5_numerical_facilities/special_functions/01_assoc_laguerre/check_nan.cc
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45250 Jakub Jelinek changed: What|Removed |Added Priority|P3 |P2 Status|WAITING |ASSIGNED CC||jakub at gcc dot gnu.org AssignedTo|unassigned at gcc dot |jakub at gcc dot gnu.org |gnu.org | --- Comment #12 from Jakub Jelinek 2010-10-26 13:25:19 UTC --- I'll retest my patch and submit. P2 as PA is now fixed.
[Bug target/45250] [4.6 Regression] FAIL: tr1/5_numerical_facilities/special_functions/01_assoc_laguerre/check_nan.cc
--- Comment #11 from dave at hiauly1 dot hia dot nrc dot ca 2010-09-21 15:16 --- Subject: Re: [4.6 Regression] FAIL: tr1/5_numerical_facilities/special_functions/01_assoc_laguerre/check_nan.cc > Hm, fixed? It's fixed on the target. I left it open because I wasn't sure whether the variable tracking issue should be addressed or not. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45250
[Bug target/45250] [4.6 Regression] FAIL: tr1/5_numerical_facilities/special_functions/01_assoc_laguerre/check_nan.cc
--- Comment #10 from rguenth at gcc dot gnu dot org 2010-09-21 13:17 --- Hm, fixed? -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45250
[Bug target/45250] [4.6 Regression] FAIL: tr1/5_numerical_facilities/special_functions/01_assoc_laguerre/check_nan.cc
--- Comment #9 from danglin at gcc dot gnu dot org 2010-09-08 23:32 --- Subject: Bug 45250 Author: danglin Date: Wed Sep 8 23:32:06 2010 New Revision: 164036 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=164036 Log: PR target/45250 * config/pa/pa.md (nonlocal_goto): Restore hard frame pointer using hard_frame_pointer_rtx instead of virtual_stack_vars_rtx. (builtin_longjmp): Likewise. (allocate_stack): Use hard_frame_pointer_rtx instead of frame_pointer_rtx. * config/pa/pa-protos.h (pa_initial_elimination_offset): Declare. * config/pa/pa.c (pa_internal_arg_pointer): Declare. (pa_can_eliminate): Likewise. (TARGET_INTERNAL_ARG_POINTER): Define. (TARGET_CAN_ELIMINATE): Define. (hppa_expand_prologue): Use hard frame pointer instead of soft frame pointer. (hppa_expand_epilogue, pa_eh_return_handler_rtx): Likewise. (pa_internal_arg_pointer, pa_can_eliminate, pa_initial_elimination_offset): New. * config/pa/pa.h (FRAME_POINTER_REGNUM): Set to new general register. (HARD_FRAME_POINTER_REGNUM): Set to register three. (INITIAL_FRAME_POINTER_OFFSET): Delete. (ELIMINABLE_REGS, INITIAL_ELIMINATION_OFFSET, DWARF_FRAME_REGISTERS): Define. (DWARF_ALT_FRAME_RETURN_COLUMN, REGNO_OK_FOR_INDEX_P, REGNO_OK_FOR_BASE_P): Update to include soft frame pointer. * config/pa/pa32-regs.h (FIRST_PSEUDO_REGISTER): Increase by one. (FIXED_REGISTERS, CALL_USED_REGISTERS, REG_ALLOC_ORDER, REG_CLASS_CONTENTS, REGNO_REG_CLASS, REGISTER_NAMES): Update to include new soft frame pointer. * config/pa/pa64-regs.h: Likewise. Modified: trunk/gcc/ChangeLog trunk/gcc/config/pa/pa-protos.h trunk/gcc/config/pa/pa.c trunk/gcc/config/pa/pa.h trunk/gcc/config/pa/pa.md trunk/gcc/config/pa/pa32-regs.h trunk/gcc/config/pa/pa64-regs.h -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45250
[Bug target/45250] [4.6 Regression] FAIL: tr1/5_numerical_facilities/special_functions/01_assoc_laguerre/check_nan.cc
--- Comment #8 from dave at hiauly1 dot hia dot nrc dot ca 2010-09-01 22:11 --- Subject: Re: [4.6 Regression] FAIL: tr1/5_numerical_facilities/special_functions/01_assoc_laguerre/check _nan.cc On Tue, 31 Aug 2010, jakub at gcc dot gnu dot org wrote: > > > --- Comment #7 from jakub at gcc dot gnu dot org 2010-08-31 19:48 --- > Created an attachment (id=21614) --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21614&action=view) > --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21614&action=view) > gcc46-pr45250.patch > > The problem is that the PA backend has quite lame setup, where > FRAME_POINTER_REGNUM is the same as HARD_FRAME_POINTER_REGNUM and > ARG_POINTER_REGNUM, thus the replacements var-tracking is doing in order to > decrease size of loclists and use DW_OP_fbreg where possible, aren't reliable, > as the testcase shows, because the same hard register can be used for > arbitrary > other data. The following patch fixes it by not doing the replacements on PA > and similar targets at all when doing VALUE based var-tracking. Ideally PA > backend would be fixed up to use different (fixed) hard regno, made up and > always eliminated, for FRAME_POINTER_REGNUM. The change fixes the PR on hppa-unknown-linux-gnu. I'll look at implementing the above this weekend. Thanks very much for looking at this problem. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45250
[Bug target/45250] [4.6 Regression] FAIL: tr1/5_numerical_facilities/special_functions/01_assoc_laguerre/check_nan.cc
--- Comment #7 from jakub at gcc dot gnu dot org 2010-08-31 19:48 --- Created an attachment (id=21614) --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21614&action=view) gcc46-pr45250.patch The problem is that the PA backend has quite lame setup, where FRAME_POINTER_REGNUM is the same as HARD_FRAME_POINTER_REGNUM and ARG_POINTER_REGNUM, thus the replacements var-tracking is doing in order to decrease size of loclists and use DW_OP_fbreg where possible, aren't reliable, as the testcase shows, because the same hard register can be used for arbitrary other data. The following patch fixes it by not doing the replacements on PA and similar targets at all when doing VALUE based var-tracking. Ideally PA backend would be fixed up to use different (fixed) hard regno, made up and always eliminated, for FRAME_POINTER_REGNUM. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45250
[Bug target/45250] [4.6 Regression] FAIL: tr1/5_numerical_facilities/special_functions/01_assoc_laguerre/check_nan.cc
--- Comment #5 from dave at hiauly1 dot hia dot nrc dot ca 2010-08-30 17:23 --- Subject: Re: [4.6 Regression] FAIL: tr1/5_numerical_facilities/special_functions/01_assoc_laguerre/check _nan.cc > Please provide preprocessed source. Sounds like my patch uncovered a latent > issue. Attached. --- Comment #6 from dave at hiauly1 dot hia dot nrc dot ca 2010-08-30 17:23 --- Created an attachment (id=21594) --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21594&action=view) -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45250
[Bug target/45250] [4.6 Regression] FAIL: tr1/5_numerical_facilities/special_functions/01_assoc_laguerre/check_nan.cc
--- Comment #4 from jakub at gcc dot gnu dot org 2010-08-30 16:06 --- Please provide preprocessed source. Sounds like my patch uncovered a latent issue. -- jakub at gcc dot gnu dot org changed: What|Removed |Added Status|UNCONFIRMED |WAITING http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45250
[Bug target/45250] [4.6 Regression] FAIL: tr1/5_numerical_facilities/special_functions/01_assoc_laguerre/check_nan.cc
-- rguenth at gcc dot gnu dot org changed: What|Removed |Added Target Milestone|--- |4.6.0 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45250
[Bug target/45250] [4.6 Regression] FAIL: tr1/5_numerical_facilities/special_functions/01_assoc_laguerre/check_nan.cc
--- Comment #3 from danglin at gcc dot gnu dot org 2010-08-27 12:50 --- Introduced in revision 162337: 2010-07-20 Jakub Jelinek * var-tracking.c (vt_expand_loc, vt_expand_loc_dummy): Bump maximum depth to 8 from 5. -- danglin at gcc dot gnu dot org changed: What|Removed |Added CC||jakub at redhat dot com http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45250
[Bug target/45250] [4.6 Regression] FAIL: tr1/5_numerical_facilities/special_functions/01_assoc_laguerre/check_nan.cc
--- Comment #2 from danglin at gcc dot gnu dot org 2010-08-19 18:33 --- (In reply to comment #1) > Introduced by following change: > 2010-07-22 Martin Jambor > PR tree-optimization/44891 > * tree-sra.c: Include gimple-pretty-print.h. > (replace_uses_with_default_def_ssa_name): Renamed to > get_repl_default_def_ssa_name, return the new SSA name instead of > replacing the old one. > (sra_modify_assign): Dump a message when removing a load, if the LHS > is an SSA_NAME, do not do any propagation, just set the RHS to a > default definition SSA NAME, type convert if necessary. > * Makefile.in (tree-sra.o): Add gimple-pretty-print.h to dependencies. Sorry Martin, I must be blind... http://gcc.gnu.org/ml/gcc-testresults/2010-08/msg01959.html -- danglin at gcc dot gnu dot org changed: What|Removed |Added CC|mjambor at suse dot cz | http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45250
[Bug target/45250] [4.6 Regression] FAIL: tr1/5_numerical_facilities/special_functions/01_assoc_laguerre/check_nan.cc
--- Comment #1 from danglin at gcc dot gnu dot org 2010-08-19 18:26 --- Introduced by following change: 2010-07-22 Martin Jambor PR tree-optimization/44891 * tree-sra.c: Include gimple-pretty-print.h. (replace_uses_with_default_def_ssa_name): Renamed to get_repl_default_def_ssa_name, return the new SSA name instead of replacing the old one. (sra_modify_assign): Dump a message when removing a load, if the LHS is an SSA_NAME, do not do any propagation, just set the RHS to a default definition SSA NAME, type convert if necessary. * Makefile.in (tree-sra.o): Add gimple-pretty-print.h to dependencies. -- danglin at gcc dot gnu dot org changed: What|Removed |Added CC||mjambor at suse dot cz http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45250
