[PATCH] D121554: [mlir][OpenMP] Added translation from `omp.atomic.capture` to LLVM IR

River Riddle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 28 17:46:20 PDT 2022


rriddle added inline comments.


================
Comment at: mlir/lib/Target/LLVMIR/Dialect/OpenMP/OpenMPToLLVMIRTranslation.cpp:1210
+    ompLoc.IP = builder.saveIP();
+    unreachableInst->removeFromParent();
+  }
----------------
This is leaking memory, remove doesn't erase the instruction.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D121554/new/

https://reviews.llvm.org/D121554



More information about the llvm-commits mailing list