[patch][pr18928] Use printable names to implement directional labels

Kevin Enderby enderby at apple.com
Thu Mar 13 10:59:32 PDT 2014


Looks good to me.
Kev

On Mar 13, 2014, at 7:57 AM, Rafael EspĂ­ndola <rafael.espindola at gmail.com> wrote:

> ping
> 
> On 24 February 2014 14:30, Rafael EspĂ­ndola <rafael.espindola at gmail.com> wrote:
>> Hi Kevin,
>> 
>> The attached patch changes the implementation of local directional
>> labels to use a dedicated map. With that it can then just use
>> CreateTempSymbol, which is what the rest of MC uses.
>> 
>> CreateTempSymbol doesn't do a great job at making sure the names are
>> unique (or being efficient when the names are not needed), but that
>> should probably be fixed in a followup patch.
>> 
>> Cheers,
>> Rafael





More information about the llvm-commits mailing list