rsmith added a comment.

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!


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

Reply via email to