ABataev added inline comments.
================
Comment at: clang/lib/Sema/SemaOpenMP.cpp:5784
Expr::EvalResult Result;
- if (CollapseLoopCountExpr->EvaluateAsInt(Result, SemaRef.getASTContext()))
+ if (!CollapseLoopCountExpr->isValueDependent() &&
+ !CollapseLoopCountExpr->isTypeDependent() &&
----------------
I would suggest to modify the code of this function if we cannot get the value
of the loops.
```
if (CollapseLoopCountExpr->isValueDependent() ||
CollapseLoopCountExpr->isTypeDependent() ||
OrderedLoopCountExpr->isValueDependent() ||
OrderedLoopCountExpr->isTypeDependent()) {
Built.clear(/* size */0);
return 1;
}
```
at the beginning of the function.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D61522/new/
https://reviews.llvm.org/D61522
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
