[PATCH] Set default to -fomit-frame-pointer
Joseph Myers
joseph@codesourcery.com
Fri Nov 3 18:04:00 GMT 2017
On Fri, 3 Nov 2017, Wilco Dijkstra wrote:
> Almost all targets add an explict -fomit-frame-pointer in the target specific
> options. Rather than doing this in a target-specific way, do this in the
Which targets do not? You should explicitly list them and CC their
maintainers and seek confirmation that such a change is appropriate for
them.
The addition of -fomit-frame-pointer through this mechanism was a
replacement for the old target macro CAN_DEBUG_WITHOUT_FP. It may now be
the cases that with DWARF debug info, having or not having a frame pointer
is not particularly relevant to debugging. But since there are other
reasons people may want a frame pointer (e.g. light-weight backtraces that
don't depend on debug / unwind info), it's at least possible there are
architecture-specific choices regarding keeping frame pointers involved
here.
--
Joseph S. Myers
joseph@codesourcery.com
More information about the Gcc-patches
mailing list