[PATCH] D138229: [flang][OpenMP] Parser support for the unroll construct (5.1)

Kiran Chandramohan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 3 10:31:48 PST 2023


kiranchandramohan accepted this revision.
kiranchandramohan added a comment.
This revision is now accepted and ready to land.

LGTM. Thanks for your patience. Please update the expectation in the test for the CI and tests to pass before submitting.



================
Comment at: flang/test/Parser/omp-unroll.f90:21
+!PARSE-TREE: OmpLoopDirective -> llvm::omp::Directive = unroll
+!PARSE-TREE: OmpClauseList -> OmpClause -> Partial -> Scalar -> Integer -> Expr = '3_4'
+!PARSE-TREE: LiteralConstant -> IntLiteralConstant = '3'
----------------
The expected value here needs an update.




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138229



More information about the llvm-commits mailing list