[llvm-branch-commits] [clang] [llvm] [mlir] [OMPIRBuilder] Add support for explicit deallocation points (PR #154752)

[Sergio Afonso via llvm-branch-commits]

================
@@ -1886,20 +1886,21 @@ void CodeGenFunction::EmitOMPParallelDirective(const OMPParallelDirective &S) {
     const CapturedStmt *CS = S.getCapturedStmt(OMPD_parallel);
     const Stmt *ParallelRegionBodyStmt = CS->getCapturedStmt();
 
-    auto BodyGenCB = [&, this](InsertPointTy AllocaIP,
-                               InsertPointTy CodeGenIP) {
+    auto BodyGenCB = [&, this](InsertPointTy AllocIP, InsertPointTy CodeGenIP,
----------------
skatrak wrote:

I did it to better reflect the semantic change, since at the same time I was adding a new argument so I thought it wouldn't introduce much noise. But it's true that doing that also meant a bunch of changes wherever the variable was used, so I applied your suggestion.

https://github.com/llvm/llvm-project/pull/154752