diff options
author | Janus Weil <janus@gcc.gnu.org> | 2013-12-16 23:01:58 +0100 |
---|---|---|
committer | Janus Weil <janus@gcc.gnu.org> | 2013-12-16 23:01:58 +0100 |
commit | 5b0b27f9c3c9efaa75187a740b8a5ed6b32a865d (patch) | |
tree | 01e39bde3ea56268f614fcc7537db764691f91dc | |
parent | 71eba7a0c684782ccf5d4befa394639f607a2717 (diff) | |
download | gcc-5b0b27f9c3c9efaa75187a740b8a5ed6b32a865d.zip gcc-5b0b27f9c3c9efaa75187a740b8a5ed6b32a865d.tar.gz gcc-5b0b27f9c3c9efaa75187a740b8a5ed6b32a865d.tar.bz2 |
re PR fortran/54949 ([F03] abstract procedure pointers not rejected)
2013-12-16 Janus Weil <janus@gcc.gnu.org>
PR fortran/54949
* symbol.c (check_conflict): Forbid abstract procedure pointers.
(gfc_add_abstract): Check for attribute conflicts.
2013-12-16 Janus Weil <janus@gcc.gnu.org>
PR fortran/54949
* gfortran.dg/proc_ptr_44.f90: New.
From-SVN: r206033
-rw-r--r-- | gcc/fortran/ChangeLog | 6 | ||||
-rw-r--r-- | gcc/fortran/symbol.c | 6 | ||||
-rw-r--r-- | gcc/testsuite/ChangeLog | 5 | ||||
-rw-r--r-- | gcc/testsuite/gfortran.dg/proc_ptr_44.f90 | 29 |
4 files changed, 45 insertions, 1 deletions
diff --git a/gcc/fortran/ChangeLog b/gcc/fortran/ChangeLog index 4f3a8e1..1a81bfc 100644 --- a/gcc/fortran/ChangeLog +++ b/gcc/fortran/ChangeLog @@ -1,3 +1,9 @@ +2013-12-16 Janus Weil <janus@gcc.gnu.org> + + PR fortran/54949 + * symbol.c (check_conflict): Forbid abstract procedure pointers. + (gfc_add_abstract): Check for attribute conflicts. + 2013-12-16 Jakub Jelinek <jakub@redhat.com> PR libgomp/59337 diff --git a/gcc/fortran/symbol.c b/gcc/fortran/symbol.c index 9d23e8b..07930f2 100644 --- a/gcc/fortran/symbol.c +++ b/gcc/fortran/symbol.c @@ -363,6 +363,7 @@ check_conflict (symbol_attribute *attr, const char *name, locus *where) *cray_pointee = "CRAY POINTEE", *data = "DATA", *value = "VALUE", *volatile_ = "VOLATILE", *is_protected = "PROTECTED", *is_bind_c = "BIND(C)", *procedure = "PROCEDURE", + *proc_pointer = "PROCEDURE POINTER", *abstract = "ABSTRACT", *asynchronous = "ASYNCHRONOUS", *codimension = "CODIMENSION", *contiguous = "CONTIGUOUS", *generic = "GENERIC"; static const char *threadprivate = "THREADPRIVATE"; @@ -593,6 +594,8 @@ check_conflict (symbol_attribute *attr, const char *name, locus *where) conf (procedure, asynchronous) conf (procedure, entry) + conf (proc_pointer, abstract) + a1 = gfc_code2string (flavors, attr->flavor); if (attr->in_namelist @@ -1440,7 +1443,8 @@ gfc_add_abstract (symbol_attribute* attr, locus* where) } attr->abstract = 1; - return true; + + return check_conflict (attr, NULL, where); } diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 7904757..671a8b5f 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2013-12-16 Janus Weil <janus@gcc.gnu.org> + + PR fortran/54949 + * gfortran.dg/proc_ptr_44.f90: New. + 2013-12-16 Jakub Jelinek <jakub@redhat.com> * c-c++-common/ubsan/overflow-mul-3.c: New test. diff --git a/gcc/testsuite/gfortran.dg/proc_ptr_44.f90 b/gcc/testsuite/gfortran.dg/proc_ptr_44.f90 new file mode 100644 index 0000000..3ed65a8 --- /dev/null +++ b/gcc/testsuite/gfortran.dg/proc_ptr_44.f90 @@ -0,0 +1,29 @@ +! { dg-do compile } +! +! PR 54949: [F03] abstract procedure pointers not rejected +! +! Contributed by Janus Weil <janus@gcc.gnu.org> + + implicit none + + abstract interface + subroutine abssub1 + end subroutine + end interface + pointer :: abssub1 ! { dg-error "PROCEDURE POINTER attribute conflicts with ABSTRACT attribute" } + + pointer :: abssub2 + abstract interface + subroutine abssub2 ! { dg-error "PROCEDURE POINTER attribute conflicts with ABSTRACT attribute" } + end subroutine + end interface + + abssub1 => sub ! { dg-error "is not a variable" } + abssub2 => sub + +contains + + subroutine sub + end subroutine + +end |