[llvm] r221960 - IR: Rewrite uniquing and creation of MDString

Duncan P. N. Exon Smith dexonsmith at apple.com
Thu Dec 4 17:43:26 PST 2014


> On 2014-Dec-04, at 17:10, Duncan P. N. Exon Smith <dexonsmith at apple.com> wrote:
> 
>> On 2014-Dec-04, at 15:44, Alexey Samsonov <vonosmas at gmail.com> wrote:
>> 
>> FYI, StringMapEntry<>::GetStringMapEntryFromValue is inherently broken - I've failed to fix it in r223402, but failed. We can't just "get the offset" of a non-standard-layout object field.
> 
> Crap.
> 
>> I'd be happy to delete GetStringMapEntryFromValue from the codebase completely, but currently MDString::getString() is its only user.
>> 
> 
> I'll remove that use, and kill the function myself.  Thanks.

r223438 and r223439.  Sorry I missed the bot failure (if there was one).





More information about the llvm-commits mailing list