Re: [PATCH v6 1/2] Kbuild: make DWARF version a choice

2021-01-29 Thread Nick Desaulniers
On Fri, Jan 29, 2021 at 1:41 PM Jakub Jelinek wrote: > > On Fri, Jan 29, 2021 at 04:32:32PM -0500, Arvind Sankar wrote: > > Given what Jakub is saying, i.e. it was previously impossible to get > > dwarf2 with gcc, and you get dwarf4 whether or not DEBUG_INFO_DWARF4 was > > It isn't impossible to g

Re: [PATCH v6 1/2] Kbuild: make DWARF version a choice

2021-01-29 Thread Jakub Jelinek
On Fri, Jan 29, 2021 at 04:32:32PM -0500, Arvind Sankar wrote: > Given what Jakub is saying, i.e. it was previously impossible to get > dwarf2 with gcc, and you get dwarf4 whether or not DEBUG_INFO_DWARF4 was It isn't impossible to get it, -gdwarf-2 works, it is just not a very good choice (at lea

Re: [PATCH v6 1/2] Kbuild: make DWARF version a choice

2021-01-29 Thread Arvind Sankar
On Fri, Jan 29, 2021 at 12:57:20PM -0800, Nick Desaulniers wrote: > On Fri, Jan 29, 2021 at 12:19 PM Nick Desaulniers > wrote: > > > > On Fri, Jan 29, 2021 at 12:17 PM Jakub Jelinek wrote: > > > > > > On Fri, Jan 29, 2021 at 11:43:17AM -0800, Nick Desaulniers wrote: > > > > Modifies CONFIG_DEBUG_

Re: [PATCH v6 1/2] Kbuild: make DWARF version a choice

2021-01-29 Thread Nick Desaulniers
On Fri, Jan 29, 2021 at 12:19 PM Nick Desaulniers wrote: > > On Fri, Jan 29, 2021 at 12:17 PM Jakub Jelinek wrote: > > > > On Fri, Jan 29, 2021 at 11:43:17AM -0800, Nick Desaulniers wrote: > > > Modifies CONFIG_DEBUG_INFO_DWARF4 to be a member of a choice. Adds an > > > explicit CONFIG_DEBUG_INFO

Re: [PATCH v6 1/2] Kbuild: make DWARF version a choice

2021-01-29 Thread Nick Desaulniers
On Fri, Jan 29, 2021 at 12:17 PM Jakub Jelinek wrote: > > On Fri, Jan 29, 2021 at 11:43:17AM -0800, Nick Desaulniers wrote: > > Modifies CONFIG_DEBUG_INFO_DWARF4 to be a member of a choice. Adds an > > explicit CONFIG_DEBUG_INFO_DWARF2, which is the default. Does so in a > > way that's forward com

Re: [PATCH v6 1/2] Kbuild: make DWARF version a choice

2021-01-29 Thread Jakub Jelinek
On Fri, Jan 29, 2021 at 11:43:17AM -0800, Nick Desaulniers wrote: > Modifies CONFIG_DEBUG_INFO_DWARF4 to be a member of a choice. Adds an > explicit CONFIG_DEBUG_INFO_DWARF2, which is the default. Does so in a > way that's forward compatible with existing configs, and makes adding > future versions

[PATCH v6 1/2] Kbuild: make DWARF version a choice

2021-01-29 Thread Nick Desaulniers
Modifies CONFIG_DEBUG_INFO_DWARF4 to be a member of a choice. Adds an explicit CONFIG_DEBUG_INFO_DWARF2, which is the default. Does so in a way that's forward compatible with existing configs, and makes adding future versions more straightforward. Suggested-by: Arvind Sankar Suggested-by: Fangrui