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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
