[PATCH] [OPENMP] Bugfix for processing of global variables in OpenMP regions.

[Alexey Bataev]

> I'm thinking we should probably split the rename of refersToEnclosingLocal -> refersToCapturedVariable into a separate patch, though.


I think we should keep it in this patch, because we're changing the meaning of this flag.


================
Comment at: lib/CodeGen/CGExpr.cpp:1905
@@ +1904,3 @@
+      else if (CapturedStmtInfo) {
+        if (auto *FD = CapturedStmtInfo->lookup(VD)) {
+          if (auto *V = LocalDeclMap.lookup(VD))
----------------
rjmccall wrote:
> It's possible to fall out of this entire block if CapturedStmtInfo->lookup(VD) fails.  Is that expected to be possible?  If not, you should have an assertion here.  If it is, I don't know what you want to do, but there should at least be a comment explaining why this is possible.
John, I fixed it.

http://reviews.llvm.org/D6259

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/