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

["Kempke, Nils-Christian via Gdb-patches" <gdb-patches@sourceware.org>]

> -----Original Message-----
> From: Andrew Burgess <aburgess@redhat.com>
> Sent: Wednesday, May 11, 2022 1:57 PM
> To: Kempke, Nils-Christian <nils-christian.kempke@intel.com>; gdb-
> patches@sourceware.org
> Cc: JiniSusan.George@amd.com; Kempke, Nils-Christian <nils-
> christian.kempke@intel.com>
> Subject: Re: [PATCH 11/18] testsuite, fortran: Add '-debug-parameters all'
> when compiling with ifx
> 
> 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.

Yes, I dropped it. Done in v2.

Thanks,
Nils

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