[llvm-branch-commits] [flang] [flang][OpenMP] Update `do concurrent` mapping pass to use `fir.do_concurrent` op (PR #138489)

[Tom Eccles via llvm-branch-commits]

================
@@ -548,19 +357,16 @@ class DoConcurrentConversionPass
       return;
     }
 
-    llvm::DenseSet<fir::DoLoopOp> concurrentLoopsToSkip;
+    llvm::DenseSet<fir::DoConcurrentOp> concurrentLoopsToSkip;
----------------
tblah wrote:

My concern was that the walk order might be changed some time because I didn't realize that was a reliable part of the API. However MLIR has so many downstream users, this is probably a theoretical concern only.

Ahh I see. This is okay for now then, because I can't think of a better solution off the top of my head either.

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