RE: [PATCH] newlib: Remove debug flag for --enable-target-optspace

"Jon Beniston" <[email protected]>
Newsgroups gmane.comp.lib.newlib
Message-ID <[email protected]>
Hi Stephanos,

>As you suggested, the debug information added by `--enable-target-optspace`
can be stripped, but why add it in the first place if it is not necessary?

Because many people want to be able to debug their code!

>Unless there is a good reason to keep `-g` and `-Os` together in
`--enable-target-optspace`, 
>I strongly believe `-g` should be removed and moved into a separate
configuration option.

Currently the default for CFLAGS_FOR_TARGET is "-g -O2".
--enable-target-optspace Changes this to "-g -Os"

If -g is to be controlled by a separate option, then it should apply in both
cases. And this new option should probably be enabled by default to maintain
existing behaviour (so perhaps add a --disable-target-debug option that
removes -g)

Cheers,
Jon
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.