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

[Patrick Lyster via Phabricator via cfe-commits]

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


Repository:
  rC Clang

https://reviews.llvm.org/D51378