mhalk added inline comments.
================ Comment at: clang/lib/CodeGen/CGStmtOpenMP.cpp:6986 + DependenceAddress = DependenciesArray.getPointer(); + CodeGenFunction::RunCleanupsScope LocalScope(*this); } ---------------- Not sure if this is useful here. Saw this `RunCleanupsScope` in conjunction with the emission of dependences + calls. But, at least in the generated IR it won't remove duplicate `depend` clauses. If anybody can comment on this, that would be great. Otherwise, I'd tend to remove it. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155692/new/ https://reviews.llvm.org/D155692 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
