On Wed, Feb 24, 2016 at 3:16 PM, Martin Liška <mli...@suse.cz> wrote:
> On 02/23/2016 04:21 PM, Richard Biener wrote:
>> On Wed, Nov 4, 2015 at 4:03 PM, Mikhail Maltsev <malts...@gmail.com> wrote:
>>> On 11/03/2015 02:35 AM, Jeff Law wrote:
>>>> This is good fore the trunk too.  Please install.
>>>>
>>>> Thanks!
>>>>
>>>> jeff
>>>
>>> Committed as r229758.
>>
>>> grep ENABLE_CHECKING *.[ch]
>> dwarf2out.c:#if ENABLE_CHECKING
>> dwarf2out.c:#if ENABLE_CHECKING
>> dwarf2out.c:#if ENABLE_CHECKING
>> dwarf2out.h:#if ENABLE_CHECKING
>
> Hi Richi.
>
> Removal in dwarf2out.c is not possible due to assignment (and read) of
> a struct member that is conditional in dwarf2out.h:
>
> struct GTY((chain_next ("%h.dw_loc_next"))) dw_loc_descr_node {
> ...
> #if ENABLE_CHECKING
>   /* When translating a function into a DWARF procedure, contains the frame
>      offset *before* evaluating this operation.  It is -1 when not yet
>      initialized.  */
>   int dw_loc_frame_offset;
> #endif
> };
>
>
>> hsa-gen.c:#ifdef ENABLE_CHECKING
>> hsa-regalloc.c:#ifdef ENABLE_CHECKING
>>> grep ENABLE_CHECKING ada/gcc-interface/*.[ch]
>> ada/gcc-interface/utils.c:#ifdef ENABLE_CHECKING
>
> I've just prepared patches for remaining files., btw. is it an acceptable 
> stage4 material?

I think so - we should then poison ENABLE_CHECKING in system.h

Richard.

> Thanks,
> Martin
>
>>
>>
>>> --
>>> Regards,
>>>     Mikhail Maltsev
>

Reply via email to