On Apr 16, 2011, at 11:40 PM, Eli Friedman wrote: > Author: efriedma > Date: Sun Apr 17 01:40:15 2011 > New Revision: 129657 > > URL: http://llvm.org/viewvc/llvm-project?rev=129657&view=rev > Log: > Hack to turn the valgrind buildbot green, until Devang can address it > properly.
Thanks! To answer Benjamin's question, this patch does not change ObjCIVar's name handling. It is captured as FieldName few lines above. The PropertyName is optional name of ObjCIVar. All other property fields are relevant only if property name is not empty. - Devang > > > Modified: > cfe/trunk/lib/CodeGen/CGDebugInfo.cpp > > Modified: cfe/trunk/lib/CodeGen/CGDebugInfo.cpp > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/CGDebugInfo.cpp?rev=129657&r1=129656&r2=129657&view=diff > ============================================================================== > --- cfe/trunk/lib/CodeGen/CGDebugInfo.cpp (original) > +++ cfe/trunk/lib/CodeGen/CGDebugInfo.cpp Sun Apr 17 01:40:15 2011 > @@ -1205,7 +1205,7 @@ > llvm::StringRef PropertyName; > llvm::StringRef PropertyGetter; > llvm::StringRef PropertySetter; > - unsigned PropertyAttributes; > + unsigned PropertyAttributes = 0; > if (ObjCPropertyDecl *PD = > ID->FindPropertyVisibleInPrimaryClass(Field->getIdentifier())) { > PropertyName = PD->getName(); > > > _______________________________________________ > cfe-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
