LGTM.
================
Comment at: lib/CodeGen/CodeGenModule.cpp:1413
@@ +1412,3 @@
+ if (const auto *VD = dyn_cast<VarDecl>(D)) {
+ EmitGlobalVarDefinition(VD);
+ if (getLangOpts().OpenMP && getOpenMPRuntime().isOMPThreadPrivateDecl(VD))
----------------
We should add a comment here explaining the separation of concerns between what
EmitGlobalVarDefinition does, what EmitOMPThreadPrivateVarDecl does, and why
they're both called here).
http://reviews.llvm.org/D4002
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits