[PATCH] D11182: [OPENMP 4.0] Initial support for 'omp declare reduction' construct.

[Richard Smith via cfe-commits]

rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.

Sorry for the delay, LGTM


================
Comment at: lib/Sema/SemaExpr.cpp:377
@@ +376,3 @@
+  auto *DRD = dyn_cast<OMPDeclareReductionDecl>(CurContext);
+  if ((LangOpts.OpenMP != 0u) && (DRD != nullptr) &&
+      !CurContext->containsDecl(D) && isa<VarDecl>(D)) {
----------------
`(LangOpts.OpenMP != 0u)` -> `LangOpts.OpenMP`
`(DRD != nullptr)` -> `DRD`

================
Comment at: lib/Sema/SemaTemplateInstantiateDecl.cpp:2494
@@ +2493,3 @@
+  auto *PrevDeclInScope = D->getPrevDeclInScope();
+  if ((PrevDeclInScope != nullptr) && !PrevDeclInScope->isInvalidDecl()) {
+    PrevDeclInScope = cast<OMPDeclareReductionDecl>(
----------------
`(PrevDeclInScope != nullptr)` -> `PrevDeclInScope`


http://reviews.llvm.org/D11182