[Openmp-commits] [PATCH] D149368: [MLIR][OpenMP] Initial Lowering of Declare Target for Data

[Slava Zakharin via Phabricator via Openmp-commits]

vzakhari added inline comments.


================
Comment at: flang/lib/Optimizer/Transforms/OMPEarlyOutlining.cpp:137
+          isAddressOfGlobalDeclareTarget(value))
+        inputs.remove(value);
 
----------------
FWIW, doesn't this invalidate the underlying iterator?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149368