diff mbox series

Warn to not add debug hook targets

Message ID nycvar.YFH.7.76.2103220938080.11502@elmra.sevgm.obk
State New
Headers show
Series Warn to not add debug hook targets | expand

Commit Message

Richard Biener March 22, 2021, 8:38 a.m. UTC 
This adds a boiler-plate warning to the debug hooks structure to
strongly discourage people from adding new debug hook targets since
we want to get rid of the current abstraction in favor of maintaining
a DWARF view of debug in the middle-end and have support for alternate
output formats to be generated off that DWARF representation.

OK?

2021-03-22  Richard Biener  <rguenther@suse.de>

	* debug.h: Add deprecation warning.
---
 gcc/debug.h | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Jeff Law March 22, 2021, 1:04 p.m. UTC | #1 
On 3/22/2021 2:38 AM, Richard Biener wrote:
> This adds a boiler-plate warning to the debug hooks structure to
> strongly discourage people from adding new debug hook targets since
> we want to get rid of the current abstraction in favor of maintaining
> a DWARF view of debug in the middle-end and have support for alternate
> output formats to be generated off that DWARF representation.
>
> OK?
>
> 2021-03-22  Richard Biener  <rguenther@suse.de>
>
> 	* debug.h: Add deprecation warning.

OK

jeff
diff mbox series

Patch

diff --git a/gcc/debug.h b/gcc/debug.h
index 67e52677b31..cd265ffbc99 100644
--- a/gcc/debug.h
+++ b/gcc/debug.h
@@ -21,6 +21,10 @@ 
 /* This structure contains hooks for the debug information output
    functions, accessed through the global instance debug_hooks set in
    toplev.c according to command line options.  */
+/* WARNING: Do not add new debug hook targets - DWARF will be the only
+   way to speak debug to the middle-end once we are able to get rid of
+   the remaining targets.  If you need alternate output formats instead
+   generate them off the DWARF representation.  */
 struct gcc_debug_hooks
 {
   /* Initialize debug output.  MAIN_FILENAME is the name of the main