ABataev added inline comments.

================
Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:8114
+      } else {
+        LValue VarLVal = CGF.EmitLValueForField(VDLVal, It->second);
+        RValue VarRVal = CGF.EmitLoadOfLValue(VarLVal, RD->getLocation());
----------------
I think you csn use `EmitLValueForFieldInitialization` just like gor the 
references and move the same code out of then-else branches because they use 
the same code. I mean, initialization of VarLVal variable can be moved out of 
the if-else.
Thise 2 functions produce the wame result for non-reference fields.


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D64558/new/

https://reviews.llvm.org/D64558



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to