ABataev added inline comments.
Comment at: clang/lib/Sema/SemaOpenMP.cpp:15356
+ Expr::EvalResult Result;
+ if (AE->getIdx()->EvaluateAsInt(Result, SemaRef.getASTContext())) {
+if (!Result.Val.getInt().isNullValue()) {
cchen wrote:
> ABataev
cchen marked an inline comment as done.
cchen added inline comments.
Comment at: clang/lib/Sema/SemaOpenMP.cpp:15356
+ Expr::EvalResult Result;
+ if (AE->getIdx()->EvaluateAsInt(Result, SemaRef.getASTContext())) {
+if (!Result.Val.getInt().isNullValue()) {
ABataev added inline comments.
Comment at: clang/lib/Sema/SemaOpenMP.cpp:15238
+ Sema
+ OpenMPClauseKind CKind;
+ OMPClauseMappableExprCommon::MappableExprComponentList
Default initializer here
Comment at:
cchen marked an inline comment as done.
cchen added inline comments.
Comment at: clang/lib/Sema/SemaOpenMP.cpp:15283
+return false;
+ return Visit(E->IgnoreParenImpCasts());;
+}
@ABataev, in the previous patch you mentioned that we don't need
cchen created this revision.
cchen added a reviewer: ABataev.
Herald added subscribers: cfe-commits, guansong.
Herald added a reviewer: jdoerfert.
Herald added a project: clang.
This step is the preparation of allowing lvalue in map/motion clause.
Repository:
rG LLVM Github Monorepo