doru1004 added inline comments.
================ Comment at: clang/lib/Sema/SemaOpenMP.cpp:1166-1168 + return llvm::any_of(Top->IteratorVarDecls, [VD](const VarDecl *IteratorVD) { + return IteratorVD == VD->getCanonicalDecl(); + }); ---------------- ABataev wrote: > doru1004 wrote: > > ABataev wrote: > > > Do you really need to store the variable in the stack, is not it enough > > > just to check that the type of this variable is BuiltinType::OMPIterator? > > I'm happy to replace this if you think it will work. Is there an example > > somewhere in the code where I can get from the VarDecl to the build in type > > you mention? > You have already a check > IteratorModifier->getType()->isSpecificBuiltinType(BuiltinType::OMPIterator), > you can you something similar for the variable This didn't work and I had to revert to using the stack! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141871/new/ https://reviews.llvm.org/D141871 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits