================
@@ -3755,10 +3759,16 @@ llvm::DIType *CGDebugInfo::CreateTypeDefinition(const 
ObjCInterfaceType *Ty,
         }
       }
     }
-    FieldTy = DBuilder.createObjCIVar(FieldName, FieldDefUnit, FieldLine,
-                                      FieldSize, FieldAlign, FieldOffset, 
Flags,
-                                      FieldTy, PropertyNode);
+    auto *IvarTy = DBuilder.createObjCIVar(FieldName, FieldDefUnit, FieldLine,
----------------
augusto2112 wrote:

Just curious, why did you need to change the variable here from `FieldTy` to 
`IvarTy`? It looks to me that keeping the `FieldTy` assignment would work.

Not that this would be better, I actually prefer the new variable, but I'd 
remove the `FieldTy = IvarTy` below and push back `IvarTy`

https://github.com/llvm/llvm-project/pull/220362
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to