aboutsummaryrefslogtreecommitdiff
path: root/gdb/testsuite/gdb.fortran/intrinsic-precedence.c
diff options
context:
space:
mode:
authorRichard Bunt <richard.bunt@linaro.org>2023-08-07 09:12:53 +0100
committerRichard Bunt <richard.bunt@linaro.org>2023-08-07 09:13:14 +0100
commit2f98b09492b33b95afcc1ac22d212ee4a5be0149 (patch)
tree70ae6d99817a818db71e0d990907db068daf6c27 /gdb/testsuite/gdb.fortran/intrinsic-precedence.c
parentad233d0d74661e5206d093e826db1eb4120c5fef (diff)
downloadfsf-binutils-gdb-2f98b09492b33b95afcc1ac22d212ee4a5be0149.zip
fsf-binutils-gdb-2f98b09492b33b95afcc1ac22d212ee4a5be0149.tar.gz
fsf-binutils-gdb-2f98b09492b33b95afcc1ac22d212ee4a5be0149.tar.bz2
gdb/fortran: Align intrinsic/variable precedence
Fortran allows variables and function to be named after language defined intrinsics as they are not reserved keywords. For example, the abs maths intrinsic can be hidden by a user declaring a variable called abs. The behavior before this patch was to favour the intrinsic, which meant that any variables named, for example "allocated", could not be inspected by GDB. This patch inverts this priority to bring GDB's behaviour closer to the Fortran language, where the user defined symbol can hide the intrinsic. Special care was need to prevent any C symbols from overriding either Fortran intrinsics or user defined variables. This was observed to be the case when GDB has access to symbols for abs from libm. This was solved by only allowing symbols not marked with language_fortran to be overridden. In total this brings the order of precedence to the following (highest first): 1. User defined Fortran variable or function. 2. Fortran intrinsic. 3. Symbols from languages other than Fortran. The sizeof intrinsic is now case insensitive. This is closer to the Fortran language. I believe this change is safe enough as it increases the acceptance of the grammar, rather than restricts it. I.e. it should not break any existing scripts which rely on it. Unless of course they rely on SIZEOF being rejected. GDB built with GCC 13. No test suite regressions detected. Compilers: GCC, ACfL, Intel, Intel LLVM, NVHPC; Platforms: x86_64, aarch64. Existing tests in gdb.fortran cover the invocation of intrinsics including: intrinsics.exp, shape.exp, rank.exp, lbound-ubound.exp. Approved-By: Tom Tromey <tom@tromey.com>
Diffstat (limited to 'gdb/testsuite/gdb.fortran/intrinsic-precedence.c')
-rw-r--r--gdb/testsuite/gdb.fortran/intrinsic-precedence.c18
1 files changed, 18 insertions, 0 deletions
diff --git a/gdb/testsuite/gdb.fortran/intrinsic-precedence.c b/gdb/testsuite/gdb.fortran/intrinsic-precedence.c
new file mode 100644
index 0000000..cec960c
--- /dev/null
+++ b/gdb/testsuite/gdb.fortran/intrinsic-precedence.c
@@ -0,0 +1,18 @@
+/* Copyright 2023 Free Software Foundation, Inc.
+
+ This program is free software; you can redistribute it and/or modify
+ it under the terms of the GNU General Public License as published by
+ the Free Software Foundation; either version 3 of the License, or
+ (at your option) any later version.
+
+ This program is distributed in the hope that it will be useful,
+ but WITHOUT ANY WARRANTY; without even the implied warranty of
+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ GNU General Public License for more details.
+
+ You should have received a copy of the GNU General Public License
+ along with this program. If not, see <http://www.gnu.org/licenses/>. */
+
+int kind (int a) {
+ return 7;
+}