[cfe-commits] r129657 - /cfe/trunk/lib/CodeGen/CGDebugInfo.cpp

Devang Patel dpatel at apple.com
Mon Apr 18 10:41:02 PDT 2011


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
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits




More information about the cfe-commits mailing list