[PATCH] D59214: [clang][OpeMP] Model OpenMP structured-block in AST (PR40563)

Bruno Ricci via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Mar 16 06:28:04 PDT 2019


riccibruno added inline comments.


================
Comment at: include/clang/AST/StmtOpenMP.h:274
+  /// Prerequisite: Executable Directive must not be Standalone directive.
+  const Stmt *getStructuredBlock() const;
 };
----------------
Uh, and what about the non-const version of `getStructuredBlock` ? I think that you just have to do the usual `const_cast` dance to implement the non-const version in term of the const version.

Apart from this I have no other comment.


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59214/new/

https://reviews.llvm.org/D59214





More information about the cfe-commits mailing list