diff mbox series

Default Alpha/VMS to DWARF2 debugging only

Message ID p7151no0-5oq4-s2nn-s762-o8on517158ss@fhfr.qr
State New
Headers show
Series Default Alpha/VMS to DWARF2 debugging only | expand

Commit Message

Richard Biener Sept. 10, 2021, 6:52 a.m. UTC 
This changes the default debug format for Alpha/VMS to DWARF2 only,
skipping emission of VMS debug info which is going do be deprecated
for GCC 12 alongside the support for STABS.

It looks like other flavors of VMS never used VMS_DEBUG by default
but only the alpha port did.

I have no good means to test anything here, it might be that we have
alpha-vms specific testcases that rely on the previous default.

OK for trunk?

Thanks,
Richard.

2021-09-10  Richard Biener  <rguenther@suse.de>

	* config/alpha/vms.h (PREFERRED_DEBUGGING_TYPE): Define to
	DWARF2_DEBUG.
---
 gcc/config/alpha/vms.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Jeff Law Sept. 10, 2021, 1:47 p.m. UTC | #1 
On 9/10/2021 12:52 AM, Richard Biener via Gcc-patches wrote:
> This changes the default debug format for Alpha/VMS to DWARF2 only,
> skipping emission of VMS debug info which is going do be deprecated
> for GCC 12 alongside the support for STABS.
>
> It looks like other flavors of VMS never used VMS_DEBUG by default
> but only the alpha port did.
>
> I have no good means to test anything here, it might be that we have
> alpha-vms specific testcases that rely on the previous default.
>
> OK for trunk?
>
> Thanks,
> Richard.
>
> 2021-09-10  Richard Biener  <rguenther@suse.de>
>
> 	* config/alpha/vms.h (PREFERRED_DEBUGGING_TYPE): Define to
> 	DWARF2_DEBUG.
It's a dead target, so yea, go for it.  Worst case it breaks someone 
notices and we know someone still cares about alpha-vms :-)

Jeff
Douglas Rupp Sept. 10, 2021, 4:12 p.m. UTC | #2 
Dwarf2 only works with an ancient GDB. VMS style was useful because it
implemented enough to get a trace back.

But as you say it’s dead, so no basis for objection.

On Fri, Sep 10, 2021 at 6:47 AM Jeff Law <jeffreyalaw@gmail.com> wrote:

>
>
> On 9/10/2021 12:52 AM, Richard Biener via Gcc-patches wrote:
> > This changes the default debug format for Alpha/VMS to DWARF2 only,
> > skipping emission of VMS debug info which is going do be deprecated
> > for GCC 12 alongside the support for STABS.
> >
> > It looks like other flavors of VMS never used VMS_DEBUG by default
> > but only the alpha port did.
> >
> > I have no good means to test anything here, it might be that we have
> > alpha-vms specific testcases that rely on the previous default.
> >
> > OK for trunk?
> >
> > Thanks,
> > Richard.
> >
> > 2021-09-10  Richard Biener  <rguenther@suse.de>
> >
> >       * config/alpha/vms.h (PREFERRED_DEBUGGING_TYPE): Define to
> >       DWARF2_DEBUG.
> It's a dead target, so yea, go for it.  Worst case it breaks someone
> notices and we know someone still cares about alpha-vms :-)
>
> Jeff
>
>
diff mbox series

Patch

diff --git a/gcc/config/alpha/vms.h b/gcc/config/alpha/vms.h
index b8673b6b6fb..2a9917cde62 100644
--- a/gcc/config/alpha/vms.h
+++ b/gcc/config/alpha/vms.h
@@ -244,7 +244,7 @@  typedef struct {int num_args; enum avms_arg_type atypes[6];} avms_arg_info;
  while (0)
 
 #undef PREFERRED_DEBUGGING_TYPE
-#define PREFERRED_DEBUGGING_TYPE VMS_AND_DWARF2_DEBUG
+#define PREFERRED_DEBUGGING_TYPE DWARF2_DEBUG
 
 #define ASM_PN_FORMAT "%s___%lu"