[PATCH] D82800: [OPENMP50] extend array section for stride (Parsing/Sema/AST)

Chi Chun Chen via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 6 12:54:29 PDT 2020


cchen marked 8 inline comments as done.
cchen added inline comments.


================
Comment at: clang/lib/Parse/ParseExpr.cpp:1933
         }
+        if (getLangOpts().OpenMP >= 50 && Tok.is(tok::colon)) {
+          // Consume ':'
----------------
ABataev wrote:
> Seems to me, it is too broad. According to the standard, it must be allowed only in to/from clauses. 
I did the check in Sema, I'll move the check here. Thanks


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D82800/new/

https://reviews.llvm.org/D82800





More information about the cfe-commits mailing list