[PATCH] D99459: [OpenMP] Implement '#pragma omp unroll'.

Michael Kruse via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 10 11:43:07 PDT 2021


Meinersbur marked an inline comment as done.
Meinersbur added inline comments.


================
Comment at: clang/lib/CodeGen/CGStmtOpenMP.cpp:2600-2604
+      RValue FactorRVal = EmitAnyExpr(FactorExpr, AggValueSlot::ignored(),
+                                      /*ignoreResult=*/true);
+      Factor =
+          cast<llvm::ConstantInt>(FactorRVal.getScalarVal())->getZExtValue();
+      assert(Factor >= 1 && "Only positive factors are valid");
----------------
ABataev wrote:
> ABataev wrote:
> > I suppose it is compiled-time expression, right? If so, use `FactorExpr->EvaluateKnownConstInt()` or something similar.
> Not done.
It was done in another branch that I forgot to merge. Sorry.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99459



More information about the cfe-commits mailing list