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

Alexey Bataev via Phabricator via Openmp-commits openmp-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 Openmp-commits mailing list