ABataev added a comment. Herald added a subscriber: jdoerfert. In D57984#1394167 <https://reviews.llvm.org/D57984#1394167>, @rsmith wrote:
> In D57984#1394067 <https://reviews.llvm.org/D57984#1394067>, @ABataev wrote: > > > In D57984#1394050 <https://reviews.llvm.org/D57984#1394050>, @rsmith wrote: > > > > > @ABataev Is it intentional that we do not propagate `Allowed` through > > > labels? For example: > > > > > > void f() { > > > #pragma omp barrier // ok > > > > > > label: > > > #pragma omp barrier // error, "cannot be an immediate substatement" > > > > > > label: > > > ; > > > #pragma omp barrier // ok > > > } > > > > > > > > > ? > > > > > > No, it is a bug. > > > Great, then I'll unify this new flag with the `Allowed` mechanism and fix the > bug as part of this change. Thanks! Sure, thanks! Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57984/new/ https://reviews.llvm.org/D57984 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
