[Bug fortran/29699] ICE in trans-decl.c
--- Comment #8 from chaoyingfu at gcc dot gnu dot org 2006-12-01 00:35 --- Subject: Bug 29699 Author: chaoyingfu Date: Fri Dec 1 00:32:38 2006 New Revision: 119391 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=119391 Log: Merged revisions 118545-118650 via svnmerge from svn+ssh://[EMAIL PROTECTED]/svn/gcc/trunk r118545 | burnus | 2006-11-07 05:27:53 -0800 (Tue, 07 Nov 2006) | 29 lines fortran/ 2006-11-06 Tobias Burnus [EMAIL PROTECTED] PR fortran/29601 * symbol.c (check_conflict, gfc_add_volatile): Add volatile support. * decl.c (match_attr_spec, gfc_match_volatile): Add volatile support. * gfortran.h (symbol_attribute): Add volatile_ to struct. * resolve.c (was_declared): Add volatile support. * trans-decl.c (gfc_finish_var_decl): Add volatile support. * match.h: Declare gfc_match_volatile. * parse.c (decode_statement): Recognize volatile. * modules.c (ab_attribute, attr_bits, mio_symbol_attribute): Add volatile support. * dump-parse-tree.c (gfc_show_attr): Add volatile support. testsuite/ 2006-11-06 Tobias Burnus [EMAIL PROTECTED] PR fortran/29601 * gfortran.dg/volatile.f90: Add. * gfortran.dg/volatile2.f90: Add. * gfortran.dg/volatile3.f90: Add. * gfortran.dg/volatile4.f90: Add. * gfortran.dg/volatile5.f90: Add. * gfortran.dg/volatile6.f90: Add. * gfortran.dg/volatile7.f90: Add. r118546 | davidu | 2006-11-07 06:08:55 -0800 (Tue, 07 Nov 2006) | 3 lines * config/mips/mips.h (ISA_HAS_PREFETCHX): Add ISA_MIPS32R2 to the list. r118547 | jiez | 2006-11-07 06:24:58 -0800 (Tue, 07 Nov 2006) | 3 lines * gcc.c (process_command): Treat -b as normal switch if its argument has no dash. r118550 | davidu | 2006-11-07 06:28:08 -0800 (Tue, 07 Nov 2006) | 5 lines * config/mips/mips.c (mips_rtx_cost_optimize_size): New table of costs when optimizing for size. (override_options): Use mips_rtx_cost_optimize_size table for cost calculations. r118552 | jules | 2006-11-07 07:16:07 -0800 (Tue, 07 Nov 2006) | 6 lines * floatformat.c (get_field): Fix segfault with little-endian word order on 64-bit hosts. (put_field): Likewise. (min): Move definition. r118553 | pault | 2006-11-07 07:17:17 -0800 (Tue, 07 Nov 2006) | 17 lines 2006-11-07 Paul Thomas [EMAIL PROTECTED] PR fortran/29539 PR fortran/29634 * decl.c (variable_decl): Add test for presence of proc_name. * error.c (gfc_error_flag_test): New function. * gfortran.h : Prototype for gfc_error_flag_test. 2006-11-07 Paul Thomas [EMAIL PROTECTED] PR fortran/29539 * gfortran.dg/gfortran.dg/blockdata_3.f90: New test. PR fortran/29634 * gfortran.dg/gfortran.dg/derived_function_interface_1.f90: New test. r118554 | bkoz | 2006-11-07 08:55:49 -0800 (Tue, 07 Nov 2006) | 49 lines 2006-11-07 Benjamin Kosnik [EMAIL PROTECTED] * include/ext/pb_ds/exception.hpp (pb_ds): Modify for -fno-exceptions. (__throw_container_error): New. Conditionalize based on __EXCEPTIONS. (__throw_insert_error): New. (__throw_join_error): New. (__throw_resize_error): New. * include/ext/pb_ds/detail/resize_policy/ hash_prime_size_policy_imp.hpp: Use them. * include/ext/pb_ds/detail/resize_policy/ hash_exponential_size_policy_imp.hpp: Same. * include/ext/pb_ds/detail/resize_policy/ hash_load_check_resize_trigger_imp.hpp: Same. * include/ext/pb_ds/detail/resize_policy/ hash_standard_resize_policy_imp.hpp: Same. * include/ext/pb_ds/detail/cc_hash_table_map_/resize_fn_imps.hpp * include/ext/pb_ds/detail/cc_hash_table_map_/ constructor_destructor_fn_imps.hpp: Same. * include/ext/pb_ds/detail/pat_trie_/insert_join_fn_imps.hpp * include/ext/pb_ds/detail/pat_trie_/split_join_branch_bag.hpp * include/ext/pb_ds/detail/pat_trie_/ constructors_destructor_fn_imps.hpp: Same. * include/ext/pb_ds/detail/bin_search_tree_/ constructors_destructor_fn_imps.hpp: Same. * include/ext/pb_ds/detail/bin_search_tree_/ split_join_fn_imps.hpp: Same. * include/ext/pb_ds/detail/gp_hash_table_map_/ insert_no_store_hash_fn_imps.hpp: Same. * include/ext/pb_ds/detail/gp_hash_table_map_/ resize_store_hash_fn_imps.hpp: Same. * include/ext/pb_ds/detail/gp_hash_table_map_/ insert_store_hash_fn_imps.hpp: Same. * include/ext/pb_ds/detail/gp_hash_table_map_/resize_fn_imps.hpp: Same. * include/ext/pb_ds/detail/gp_hash_table_map_/ constructor_destructor_fn_imps.hpp: Same. * include/ext/pb_ds/detail/gp_hash_table_map_/ resize_no_store_hash_fn_imps.hpp: Same.
[Bug fortran/29699] ICE in trans-decl.c
--- Comment #6 from pault at gcc dot gnu dot org 2006-11-12 07:40 --- Subject: Bug 29699 Author: pault Date: Sun Nov 12 07:40:26 2006 New Revision: 118719 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=118719 Log: 2006-11-12 Paul Thomas [EMAIL PROTECTED] PR fortran/29699 * trans-array.c (structure_alloc_comps): Detect pointers to arrays and use indirect reference to declaration. * resolve.c (resolve_fl_variable): Tidy up condition. (resolve_symbol): The same and only add initialization code if the symbol is referenced. * trans-decl.c (gfc_trans_deferred_vars): Call gfc_trans_ deferred_array before gfc_trans_auto_array_allocation. PR fortran/21730 * symbol.c (check_done): Remove. (gfc_add_attribute): Remove reference to check_done and remove the argument attr_intent. (gfc_add_allocatable, gfc_add_dimension, gfc_add_external, gfc_add_intrinsic, gfc_add_optional, gfc_add_pointer, gfc_add_cray_pointer, gfc_add_cray_pointee, gfc_add_result, gfc_add_target, gfc_add_in_common, gfc_add_elemental, gfc_add_pure, gfc_add_recursive, gfc_add_procedure, gfc_add_type): Remove references to check_done. * decl.c (attr_decl1): Eliminate third argument in call to gfc_add_attribute. * gfortran.h : Change prototype for gfc_add_attribute. PR fortran/29431 * trans-array.c(get_array_ctor_strlen): If we fall through to default, use a constant character length if it is available. PR fortran/29758 * check.c (gfc_check_reshape): Check that there are enough elements in the source array as to be able to fill an array defined by shape, when pad is absent. PR fortran/29315 * trans-expr.c (is_aliased_array): Treat correctly the case where the component is itself and array or array reference. 2006-11-12 Paul Thomas [EMAIL PROTECTED] PR fortran/29699 * gfortran.dg/alloc_comp_auto_array_1.f90: New test. PR fortran/21730 * gfortran.dg/change_symbol_attributes_1.f90: New test. PR fortran/29431 * gfortran.dg/array_constructor_13.f90: New test. PR fortran/29758 * gfortran.dg/reshape_source_size_1.f90: New test. PR fortran/29315 * gfortran.dg/aliasing_dummy_4.f90: New test. Added: branches/gcc-4_2-branch/gcc/testsuite/gfortran.dg/aliasing_dummy_4.f90 branches/gcc-4_2-branch/gcc/testsuite/gfortran.dg/alloc_comp_auto_array_1.f90 branches/gcc-4_2-branch/gcc/testsuite/gfortran.dg/array_constructor_13.f90 branches/gcc-4_2-branch/gcc/testsuite/gfortran.dg/change_symbol_attributes_1.f90 branches/gcc-4_2-branch/gcc/testsuite/gfortran.dg/reshape_source_size_1.f90 Modified: branches/gcc-4_2-branch/gcc/fortran/ChangeLog branches/gcc-4_2-branch/gcc/fortran/check.c branches/gcc-4_2-branch/gcc/fortran/decl.c branches/gcc-4_2-branch/gcc/fortran/gfortran.h branches/gcc-4_2-branch/gcc/fortran/resolve.c branches/gcc-4_2-branch/gcc/fortran/symbol.c branches/gcc-4_2-branch/gcc/fortran/trans-array.c branches/gcc-4_2-branch/gcc/fortran/trans-decl.c branches/gcc-4_2-branch/gcc/fortran/trans-expr.c branches/gcc-4_2-branch/gcc/testsuite/ChangeLog -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=29699
[Bug fortran/29699] ICE in trans-decl.c
--- Comment #7 from pault at gcc dot gnu dot org 2006-11-12 07:41 --- Fixed on trunk and 4.2. 4.1 will follow next weekend. Paul -- pault at gcc dot gnu dot org changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution||FIXED http://gcc.gnu.org/bugzilla/show_bug.cgi?id=29699
[Bug fortran/29699] ICE in trans-decl.c
--- Comment #5 from pault at gcc dot gnu dot org 2006-11-09 18:42 --- Subject: Bug 29699 Author: pault Date: Thu Nov 9 18:42:28 2006 New Revision: 118624 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=118624 Log: 2006-11-09 Paul Thomas [EMAIL PROTECTED] PR fortran/29699 * trans-array.c (structure_alloc_comps): Detect pointers to arrays and use indirect reference to declaration. * resolve.c (resolve_fl_variable): Tidy up condition. (resolve_symbol): The same and only add initialization code if the symbol is referenced. * trans-decl.c (gfc_trans_deferred_vars): Call gfc_trans_ deferred_array before gfc_trans_auto_array_allocation. PR fortran/21730 * symbol.c (check_done): Remove. (gfc_add_attribute): Remove reference to check_done and remove the argument attr_intent. (gfc_add_allocatable, gfc_add_dimension, gfc_add_external, gfc_add_intrinsic, gfc_add_optional, gfc_add_pointer, gfc_add_cray_pointer, gfc_add_cray_pointee, gfc_add_result, gfc_add_target, gfc_add_in_common, gfc_add_elemental, gfc_add_pure, gfc_add_recursive, gfc_add_procedure, gfc_add_type): Remove references to check_done. * decl.c (attr_decl1): Eliminate third argument in call to gfc_add_attribute. * gfortran.h : Change prototype for gfc_add_attribute. 2006-11-09 Paul Thomas [EMAIL PROTECTED] PR fortran/29699 * gfortran.dg/alloc_comp_auto_array_1.f90: New test. PR fortran/21730 * gfortran.dg/change_symbol_attributes_1.f90: New test. Added: trunk/gcc/testsuite/gfortran.dg/alloc_comp_auto_array_1.f90 trunk/gcc/testsuite/gfortran.dg/change_symbol_attributes_1.f90 Modified: trunk/gcc/fortran/ChangeLog trunk/gcc/fortran/decl.c trunk/gcc/fortran/gfortran.h trunk/gcc/fortran/resolve.c trunk/gcc/fortran/symbol.c trunk/gcc/fortran/trans-array.c trunk/gcc/fortran/trans-decl.c trunk/gcc/testsuite/ChangeLog -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=29699
[Bug fortran/29699] ICE in trans-decl.c
--- Comment #4 from patchapp at dberlin dot org 2006-11-07 10:30 --- Subject: Bug number PR29699 A patch for this bug has been added to the patch tracker. The mailing list url for the patch is http://gcc.gnu.org/ml/gcc-patches/2006-11/msg00393.html -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=29699
[Bug fortran/29699] ICE in trans-decl.c
--- Comment #3 from pault at gcc dot gnu dot org 2006-11-06 19:14 --- Created an attachment (id=12552) -- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=12552action=view) Patch to fix PR and two further, hidden problems The comments in the testcase tell the story. I need to think about deallocation of allocatable components overnight, before I submit. There is something not quite right there. With this patch, automatic arrays of structures with allocatable components are usable. Paul -- pault at gcc dot gnu dot org changed: What|Removed |Added AssignedTo|unassigned at gcc dot gnu |pault at gcc dot gnu dot org |dot org | Status|NEW |ASSIGNED http://gcc.gnu.org/bugzilla/show_bug.cgi?id=29699
[Bug fortran/29699] ICE in trans-decl.c
--- Comment #2 from pault at gcc dot gnu dot org 2006-11-05 22:03 --- We can move into different territory by referring to vocab_swap: PROGRAM vocabulary_word_count IMPLICIT NONE TYPE VARYING_STRING CHARACTER,DIMENSION(:),ALLOCATABLE :: chars ENDTYPE VARYING_STRING INTEGER :: list_size=200 CONTAINS SUBROUTINE extend_lists type(VARYING_STRING),DIMENSION(list_size) :: vocab_swap allocate (vocab_swap(1)%chars(10)) ENDSUBROUTINE extend_lists ENDPROGRAM vocabulary_word_count bombs out, as follows: pr29699.f90: In function 'extend_lists': pr29699.f90:11: internal compiler error: tree check: expected record_type or uni on_type or qual_union_type, have pointer_type in find_compatible_field, at tree. c:7108 Please submit a full bug report, So I would say that there are two bugs! It is interesting to compare the code produced here with taht produced when list_size is made a parameter. This appears to be the clue but it is a bit opaque right now. Paul -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=29699
[Bug fortran/29699] ICE in trans-decl.c
--- Comment #1 from eedelman at gcc dot gnu dot org 2006-11-03 23:20 --- Confirmed. -- eedelman at gcc dot gnu dot org changed: What|Removed |Added CC||eedelman at gcc dot gnu dot ||org Status|UNCONFIRMED |NEW Ever Confirmed|0 |1 Last reconfirmed|-00-00 00:00:00 |2006-11-03 23:20:50 date|| http://gcc.gnu.org/bugzilla/show_bug.cgi?id=29699
