[PATCH] D57615: [AST][OpenMP] OpenMP master Construct contains Structured block

Alexey Bataev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 1 13:07:25 PST 2019


ABataev added inline comments.


================
Comment at: lib/Sema/SemaOpenMP.cpp:6058
+  // longjmp() and throw() must not violate the entry/exit criteria.
+  cast<CapturedStmt>(AStmt)->getCapturedDecl()->setNothrow();
+
----------------
Generally speaking, this is not required. It is required only for those constructs, that really needs an outlined function. Master construct does not create outlined function, so the "nothrow" flag is useless here.


Repository:
  rC Clang

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

https://reviews.llvm.org/D57615





More information about the cfe-commits mailing list