estewart08 added inline comments. Herald added subscribers: sstefan1, yaxunl. Herald added a project: All.
================ Comment at: clang/lib/Sema/SemaOpenMP.cpp:15450 } + bool VisitCXXThisExpr(CXXThisExpr *CTE) { return true; } + bool VisitStmt(Stmt *) { ---------------- cchen wrote: > ABataev wrote: > > Do you really need this function? > Removed the function. Was this function intended to be removed? As far as I can tell it was not and it seems to be the source of an issue I am having: expected addressable lvalue in 'map' clause Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D74970/new/ https://reviews.llvm.org/D74970 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits