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

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 1 13:25:17 PST 2019


lebedev.ri added a comment.

In D57615#1381418 <https://reviews.llvm.org/D57615#1381418>, @ABataev wrote:

> In D57615#1381416 <https://reviews.llvm.org/D57615#1381416>, @lebedev.ri wrote:
>
> > @ABataev i'm not sure i have fully followed the https://bugs.llvm.org/show_bug.cgi?id=40563#c1
> >
> > > The outlined function is not generated for the loop, so there is no problem with the standard compatibility.
> >
> > Are you saying that in these cases of `master`, `critical`, `single` directives, `CapturedDecl` should **not** have `nothrow` bit set too?
>
>
> This flag just does not matter for them.


I'll rephrase:
Are you opposed to providing the correct (as per the specification) knowledge that
no exception will escape out of these `CapturedDecl`'s, because that knowledge
does not matter for the existing sema/codegen, and does not affect produced IR?


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