Re: [PATCH 11/18] testsuite, fortran: Add '-debug-parameters all' when compiling with ifx

[Andrew Burgess via Gdb-patches <gdb-patches@sourceware.org>]

Nils-Christian Kempke <nils-christian.kempke@intel.com> writes:

> In order for ifx to emit all debug entries, even for unused parameters
> in modules we have to define the '-debug-parameters all' flag.
>
> This commit adds it to the ifx-* specific flags in gdb.exp.
> ---
>  gdb/testsuite/lib/gdb.exp | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/gdb/testsuite/lib/gdb.exp b/gdb/testsuite/lib/gdb.exp
> index b8e61d846d..e799312c12 100644
> --- a/gdb/testsuite/lib/gdb.exp
> +++ b/gdb/testsuite/lib/gdb.exp
> @@ -4439,6 +4439,8 @@ proc gdb_compile {source dest type options} {
>      # Place (and look for) Fortran `.mod` files in the output
>      # directory for this specific test.  For Intel compilers the -J
>      # option is not supported so instead use the -module flag.
> +    # Additionally, ifx needs the -debug-parameters all flag set to emit

I think you should place quotes around the "-debug-parameters all" to
indicate that the "all" is part of the flag, or, just drop the "all", we
can see what -debug-parameters is set too by looking below.

This is fine with that change.

Thanks,
Andrew


> +    # debug info for all parameters in modules.
>      if { $getting_compiler_info == 0 && [lsearch -exact $options f90] != -1 } {
>  	# Fortran compile.
>  	set mod_path [standard_output_file ""]
> @@ -4447,6 +4449,9 @@ proc gdb_compile {source dest type options} {
>  	} elseif { [test_compiler_info {ifort-*} f90]
>  		   || [test_compiler_info {ifx-*} f90] } {
>  	    lappend new_options "additional_flags=-module ${mod_path}"
> +	    if { [test_compiler_info {ifx-*} f90] } {
> +		lappend new_options "additional_flags=-debug-parameters all"
> +	    }
>  	}
>      }
>  
> -- 
> 2.25.1
>
> Intel Deutschland GmbH
> Registered Address: Am Campeon 10, 85579 Neubiberg, Germany
> Tel: +49 89 99 8853-0, www.intel.de <http://www.intel.de>
> Managing Directors: Christin Eisenschmid, Sharon Heck, Tiffany Doon Silva  
> Chairperson of the Supervisory Board: Nicole Lau
> Registered Office: Munich
> Commercial Register: Amtsgericht Muenchen HRB 186928