ABataev 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(); + }); ---------------- doru1004 wrote: > 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! Why? 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