[PATCH] Remove MAY_HAVE_DEBUG_MARKER_STMTS and MAY_HAVE_DEBUG_BIND_STMTS.

Jeff Law jeffreyalaw@gmail.com
Tue Nov 16 02:21:11 GMT 2021



On 11/12/2021 7:37 AM, Martin Liška wrote:
> @Alexandre: PING
>
> On 10/18/21 12:05, Richard Biener wrote:
>> On Mon, Oct 18, 2021 at 10:54 AM Martin Liška <mliska@suse.cz> wrote:
>>>
>>> The macros correspond 1:1 to an option flags and make it harder
>>> to find all usages of the flags.
>>>
>>> Patch can bootstrap on x86_64-linux-gnu and survives regression tests.
>>>
>>> Ready to be installed?
>>
>> Hmm, they were introduced on purpose - since you leave around
>> MAY_HAVE_DEBUG_STMTS they conceptually make the code
>> easier to understand.
>>
>> So I'm not sure if we want this change.  CCed Alex so maybe he
>> can weight in.
I'd give it another 48hrs, then go ahead and commit.  My recollection is 
those were in place to allow the bulk of the work to go in independent 
of the switches that activated the SFN debugging bits.


Jeff


More information about the Gcc-patches mailing list