Re: [PATCH 1/3] STABS: remove -gstabs and -gxcoff functionality

2022-11-10 Thread Martin Liška
On 11/10/22 15:12, Michael Matz wrote: Hello, On Thu, 10 Nov 2022, Martin Liška wrote: These changes are part of commit r13-2361-g7e0db0cdf01e9c885a29cb37415f5bc00d90c029 "STABS: remove -gstabs and -gxcoff functionality". What this does is remove these identifiers from "poisoning": /*

Re: [PATCH 1/3] STABS: remove -gstabs and -gxcoff functionality

2022-11-10 Thread Michael Matz via Gcc-patches
Hello, On Thu, 10 Nov 2022, Martin Liška wrote: > > These changes are part of > > commit r13-2361-g7e0db0cdf01e9c885a29cb37415f5bc00d90c029 > > "STABS: remove -gstabs and -gxcoff functionality". What this does is > > remove these identifiers from "poisoning": > > > > /* As the last action

Re: [PATCH 1/3] STABS: remove -gstabs and -gxcoff functionality

2022-11-10 Thread Martin Liška
On 11/4/22 10:32, Thomas Schwinge wrote: Hi! On 2022-09-01T12:05:23+0200, Martin Liška wrote: gcc/ChangeLog: --- a/gcc/system.h +++ b/gcc/system.h @@ -1009,8 +1009,7 @@ extern void fancy_abort (const char *, int, const char *) ASM_OUTPUT_DEFINE_LABEL_DIFFERENCE_SYMBOL HOST_WORDS_BIG_

Re: [PATCH 1/3] STABS: remove -gstabs and -gxcoff functionality

2022-11-04 Thread Thomas Schwinge
Hi! On 2022-09-01T12:05:23+0200, Martin Liška wrote: > gcc/ChangeLog: > --- a/gcc/system.h > +++ b/gcc/system.h > @@ -1009,8 +1009,7 @@ extern void fancy_abort (const char *, int, const char > *) > ASM_OUTPUT_DEFINE_LABEL_DIFFERENCE_SYMBOL HOST_WORDS_BIG_ENDIAN\ > OBJC_PROLOGUE

Re: [PATCH 1/3] STABS: remove -gstabs and -gxcoff functionality

2022-09-07 Thread Martin Liška
On 9/6/22 19:00, David Edelsohn wrote: > * dwarf2out.cc (XCOFF_DEBUGGING_INFO): Likewise. > (HAVE_XCOFF_DWARF_EXTRAS): Likewise. > (output_fde): Likewise. > (output_call_frame_info): Likewise. > (have_macinfo): Likewise. > (add_AT_loc_list): Likewise. > (ad

Re: [PATCH 1/3] STABS: remove -gstabs and -gxcoff functionality

2022-09-06 Thread David Edelsohn via Gcc-patches
* dwarf2out.cc (XCOFF_DEBUGGING_INFO): Likewise. (HAVE_XCOFF_DWARF_EXTRAS): Likewise. (output_fde): Likewise. (output_call_frame_info): Likewise. (have_macinfo): Likewise. (add_AT_loc_list): Likewise. (add_AT_view_list): Likewise. (out

Re: [PATCH 1/3] STABS: remove -gstabs and -gxcoff functionality

2022-09-06 Thread David Edelsohn via Gcc-patches
I fully support the plan to remove stabs support, but this patch broke bootstrap on AIX. It seems rather bad policy to remove support for a feature without ensuring that the removal does not negatively impact the targets touched by the patch. I should have been explicitly copied on these patches

Re: [PATCH 1/3] STABS: remove -gstabs and -gxcoff functionality

2022-09-05 Thread Richard Biener via Gcc-patches
On Mon, Sep 5, 2022 at 9:59 AM Martin Liška wrote: > > On 9/2/22 10:54, Richard Biener wrote: > > On Fri, Sep 2, 2022 at 9:00 AM Martin Liška wrote: > >> > >> On 9/1/22 13:18, Richard Biener wrote: > >>> I presume WarnRemoved will diagnose use of -gstabs but not fail > >>> compilation. Will -gst

Re: [PATCH 1/3] STABS: remove -gstabs and -gxcoff functionality

2022-09-05 Thread Martin Liška
On 9/2/22 10:54, Richard Biener wrote: > On Fri, Sep 2, 2022 at 9:00 AM Martin Liška wrote: >> >> On 9/1/22 13:18, Richard Biener wrote: >>> I presume WarnRemoved will diagnose use of -gstabs but not fail >>> compilation. Will -gstabs then still enable -g (with the default debug >>> format)? >> >

Re: [PATCH 1/3] STABS: remove -gstabs and -gxcoff functionality

2022-09-02 Thread Richard Biener via Gcc-patches
On Fri, Sep 2, 2022 at 9:00 AM Martin Liška wrote: > > On 9/1/22 13:18, Richard Biener wrote: > > I presume WarnRemoved will diagnose use of -gstabs but not fail > > compilation. Will -gstabs then still enable -g (with the default debug > > format)? > > No, it won't set -g option. That was the u

Re: [PATCH 1/3] STABS: remove -gstabs and -gxcoff functionality

2022-09-02 Thread Martin Liška
On 9/1/22 13:18, Richard Biener wrote: I presume WarnRemoved will diagnose use of -gstabs but not fail compilation. Will -gstabs then still enable -g (with the default debug format)? No, it won't set -g option. Please followup with a gcc-13/changes.html entry. Sure. I notice we have VM