ABataev added inline comments.
================ Comment at: clang/include/clang/AST/StmtOpenMP.h:291 + /// the frontend. + OpenMPDirectiveKind PrevMappedDirective = llvm::omp::OMPD_unknown; + ---------------- I don't see where this field is stored/loaded for PCH support. You need add a support for this in Serialization. ================ Comment at: clang/include/clang/AST/StmtOpenMP.h:611-614 + void setMappedDirective(OpenMPDirectiveKind MappedDirective) { + PrevMappedDirective = MappedDirective; + } + ---------------- Better to make it a part of Create member function rather than having a separate function for this. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D144634/new/ https://reviews.llvm.org/D144634 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits