This is the mail archive of the newlib@sourceware.org mailing list for the newlib project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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

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
Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]