[llvm-commits] [llvm] r135042 - /llvm/trunk/lib/VMCore/Type.cpp

Chris Lattner clattner at apple.com
Wed Jul 13 12:53:12 PDT 2011


On Jul 13, 2011, at 11:44 AM, Jay Foad wrote:

>> I don't think this patch is doing anything.  Previously we had:
>> 
>>  StructType *&ST = Context.pImpl->AnonStructTypes[Key];
>> 
>>  if (ST) return ST;
>> 
>>  // Value not found.  Create a new type!
>>  ST = new StructType(Context);
>> 
>> 
>> Because "ST" is a reference into the map, the assignment is actually adding the mapping.
> 
> Right. Sorry. I'll back it out.

Thanks! No problem,

-Chris



More information about the llvm-commits mailing list