From 8d2ef06e1c220bcfb133a47b98b6287ccabdb587 Mon Sep 17 00:00:00 2001 From: Nils-Christian Kempke Date: Mon, 17 Jan 2022 12:30:05 +0100 Subject: [PATCH] gdb, testsuite, fortran: adapt info symbol expected output for intel compilers Info symbol is expected to print the symbol table name of a symbol, since symbol lookup happens via the minimal symbol table. This name corresponds to the linkage name in the full symbol table. For gfortran (and maybe others) these names currently have the form XXXX.NUMBER where XXXX is the symbol name and NUMBER a compiler generated appendix for mangling. An example taken from the modified nested-funcs-2.exp would be ~~~~ $ objdump -t ./outputs/gdb.fortran/nested-funcs-2/nested-funcs-2 | grep \ increment 00000000000014ab l F .text 0000000000000095 increment.3883 000000000000141c l F .text 000000000000008f increment_program_global.3881 ~~~~ This mangled name gets recognized by the Ada demangler/decoder and decoded as Ada to XXXX (setting the symbol language to Ada). This leads to output of XXXX over XXXX.NUMBER for info symbol on gfortran symbols. For ifort and ifx the generated linkage names have the form SCOPEA_SCOPEB_XXXX_ which are not recognized by the Ada decoder (or any other demangler for that matter) and thus printed as is. The respective objdump in the above case looks like ~~~~ $ objdump -t ./outputs/gdb.fortran/nested-funcs-2/nested-funcs-2 | grep \ increment 0000000000403a44 l F .text 0000000000000074 contains_keyword_IP_increment_ 0000000000403ab8 l F .text 0000000000000070 contains_keyword_IP_increment_program_global_ ~~~~ In the unmodified testcase this results in 'fails' when ran with the intel compilers: ~~~~ >> make check RUNTESTFLAGS="gdb.fortran/nested-funcs-2.exp \ GDBFLAGS='$GDBFLAGS' CC_FOR_TARGET='icpc' F90_FOR_TARGET='ifort'" ... === gdb Summary === \# of expected passes 80 \# of unexpected failures 14 ~~~~ Note that there is no Fortran mangling standard. We keep the gfortran behavior as is and modify the test to reflect ifx and ifort mangled names which fixes above fails. Signed-off-by: Nils-Christian Kempke --- gdb/testsuite/gdb.fortran/nested-funcs-2.exp | 29 ++++++++++++++++++-- 1 file changed, 27 insertions(+), 2 deletions(-) diff --git a/gdb/testsuite/gdb.fortran/nested-funcs-2.exp b/gdb/testsuite/gdb.fortran/nested-funcs-2.exp index d04fd0987e2..6009cb65c53 100644 --- a/gdb/testsuite/gdb.fortran/nested-funcs-2.exp +++ b/gdb/testsuite/gdb.fortran/nested-funcs-2.exp @@ -50,6 +50,31 @@ proc do_bp_tests {with_src_prefix_p with_nest_prefix_p} { set nest_prefix "" } + # Normally, info symbol prints the symbol table name for any fortran + # symbols (since symbol lookup happens via the minimal symbol + # table). This would correspond to the linkage name in the full symbol + # table. + # For gfortran (and maybe others) these names currently have the form + # XXXX.NUMBER where XXXX is the symbol name and NUMBER a compiler generated + # appendix for mangling. This mangled name gets recognized by the Ada + # demangler/decoder and decoded as Ada (setting the symbol language to Ada) + # to XXXX. This leads to the somewhat unexpected output of XXXX over + # XXXX.NUMBER for info symbol. + # For ifort and ifx the generated linkage names have the form + # SCOPEA_SCOPEB_XXXX_ which is not recognized by the Ada demangler and thus + # printed as is. + # Note that there is no Fortran mangling standard. We keep the + # gfortran behavior as is and extend the test to reflect ifx and ifort + # mangling. + proc get_linkage_name_pattern {symbol_name} { + + if { [test_compiler_info icc*] || [test_compiler_info intel*]} { + return "\(?:.*_\)?${symbol_name}_?" + } else { + return ${symbol_name} + } + } + # Test setting up breakpoints and otherwise examining nested # functions before the program starts. with_test_prefix "before start" { @@ -68,7 +93,7 @@ proc do_bp_tests {with_src_prefix_p with_nest_prefix_p} { # is a failure, just a limitation in current GDB. if { ${with_nest_prefix_p} } { gdb_test "info symbol ${nest_prefix}${function}" \ - "${function} in section .*" + "[get_linkage_name_pattern ${function}] in section .*" gdb_test "whatis ${nest_prefix}${function}" \ "type = ${type}" gdb_test "ptype ${nest_prefix}${function}" \ @@ -134,7 +159,7 @@ proc do_bp_tests {with_src_prefix_p with_nest_prefix_p} { set type [lindex $entry 1] with_test_prefix $function { gdb_test "info symbol ${nest_prefix}$function" \ - "$function in section .*" + "[get_linkage_name_pattern $function] in section .*" gdb_test "whatis ${nest_prefix}$function" \ "type = ${type}" gdb_test "ptype ${nest_prefix}$function" \ -- 2.30.2