[PATCH] D91944: OpenMP 5.0 metadirective

Joachim Protze via Phabricator via cfe-commits cfe-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 cfe-commits mailing list