On 09/15/2015 04:11 AM, Richard Biener wrote:
I wouldn't expect to see any in the .c files. There'll likely be
stragglers (like tree/rtl checking). But even flushing all the stuff out of
the .c files is a major win.
Do you have a preference on whether or not to make flag_checking a pre, post
or integrated patch? I can see arguments for any of those three choices.
I'd like to see it as part of the patch, even if the initial
implementation would be a
static const int flag_checking = CHECKING_P;
in flags.h so we can avoid chaging all uses twice. It requires some
more careful
review of what uses are changed to flag_checking and which one to CHECKING_P
of course which means it would make sense to do it as a pre patch handling the
obvious cases first. And then adding -fchecking "properly" wouldn't be too hard
anyway.
That works for me. Mikhail, can you move things in that direction?
jeff
