[patch][pr17918] Use simple section names for COMDAT sections

Rafael EspĂ­ndola rafael.espindola at gmail.com
Tue Nov 26 15:23:18 PST 2013


sorry, just the wrong list.

On 26 November 2013 18:14, Richard Smith <richard at metafoo.co.uk> wrote:
> On Tue, Nov 26, 2013 at 2:57 PM, Rafael EspĂ­ndola
> <rafael.espindola at gmail.com> wrote:
>>
>> With the attached patch we use simple names for COMDAT sections (like
>> .text or .bss). This matches the MSVC behavior.
>>
>> When merging it is the COMDAT symbol that is used to decide if two
>> sections should be merged, so there is no point in building a fancy
>> name.
>>
>> This survived a bootstrap on mingw32.
>
>
> What sort of response are you looking for here? This isn't the right list
> for a review, and the patch seems not ready for commit; do you want people
> to test this on other platforms?




More information about the cfe-commits mailing list