[PATCH] D51378: [OPENMP] Add support for nested 'declare target' directives

Patrick Lyster via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 30 08:33:08 PDT 2018

patricklyster added a comment.

In https://reviews.llvm.org/D51378#1218184, @RaviNarayanaswamy wrote:

> We should just go with generating an error if the DeclareTargetNestingLevel is not 0 at the end of compilation unit.  
>  Hard to detect if user accidentally forgot to have end declare in header file and had it in the include file or it was intentional.

This is the current behaviour

  rC Clang


More information about the cfe-commits mailing list