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

Patrick Lyster via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 5 07:33:46 PDT 2018


patricklyster updated this revision to Diff 164037.
patricklyster added a comment.

Previous implementation only supported immediately nested declare targets such as below:

  #pragma omp declare target
    #pragma omp declare target
      int foo();
    #pragma omp end declare target
    int bar();
  #pragma omp end declare target

Update allows for nesting in this format:

  #pragma omp declare target
    int foo();
    #pragma omp declare target
      int bar();
    #pragma omp end declare target
  #pragma omp end declare target


https://reviews.llvm.org/D51378

Files:
  clang/include/clang/Basic/DiagnosticSemaKinds.td
  clang/include/clang/Sema/Sema.h
  clang/lib/Parse/ParseOpenMP.cpp
  clang/lib/Sema/SemaOpenMP.cpp
  clang/test/OpenMP/Inputs/declare_target_include.h
  clang/test/OpenMP/declare_target_ast_print.cpp
  clang/test/OpenMP/declare_target_messages.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D51378.164037.patch
Type: text/x-patch
Size: 6930 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180905/70b7a365/attachment-0001.bin>


More information about the cfe-commits mailing list