[PATCH 05/11] Handle const and varible modifiers for CodeView

Jeff Law jeffreyalaw@gmail.com
Tue Jun 25 05:42:02 GMT 2024



On 6/24/24 8:49 PM, Mark Harmstone wrote:
> On 24/6/24 04:39, Jeff Law wrote:
>>>
>> So presumably you're freeing these objects elsewhere?  I see the free 
>> (custom_types), but I don' see where you free an subobjects.  Did I 
>> miss something?
>>
>> I'll go ahead and commit, but please double check for memory leaks.
> 
> Thanks Jeff. I just realized I wrote "varible" rather than "volatile" - 
> ah well.
Trivially fixable with a follow-up patch.

> 
> See patch 4 - write_custom_types loops through the custom_types linked 
> list, and removes and frees the head until it's empty.
Thanks.  I suspected it was the walk down that list, but better to just 
ask the author to be sure :-)  Thanks.

jeff



More information about the Gcc-patches mailing list