[PATCH] D91944: OpenMP 5.0 metadirective
    Joachim Protze via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Jan 18 08:21:37 PST 2021
    
    
  
protze.joachim added a comment.
I found two issues with this patch regarding the `default` clause:
- The spec does not require a default clause. I get `error: expected expression` if I omit a default clause. The error is gone if I add `default()`.
- The spec does not allow an empty `default()` clause, but rather expects `default(nothing)`or omission of the default clause. This patch accepts the empty default clause.
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D91944/new/
https://reviews.llvm.org/D91944
    
    
More information about the llvm-commits
mailing list