ABataev added inline comments.
================ Comment at: clang/lib/Sema/SemaExpr.cpp:4907 + if (Stride) { + auto Res = + PerformOpenMPImplicitIntegerConversion(Stride->getExprLoc(), Stride); ---------------- cchen wrote: > ABataev wrote: > > `auto`->`ExprResult` > Should I also change it for LowerBound and Length? Just to make it consistent. Not in this patch Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D82800/new/ https://reviews.llvm.org/D82800 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits