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

Ravi Narayanaswamy via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 29 13:04:20 PDT 2018


RaviNarayanaswamy added a comment.

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.


Repository:
  rC Clang

https://reviews.llvm.org/D51378





More information about the cfe-commits mailing list