[llvm-branch-commits] [mlir] [mlir][omp] Improve canonloop/iv naming (PR #159773)

[Michael Kruse via llvm-branch-commits]

================
@@ -77,6 +77,177 @@ struct LLVMPointerPointerLikeModel
 };
 } // namespace
 
+/// Generate a name of a canonical loop nest of the format
+/// `<prefix>(_s<num>_r<num>)*` that describes its nesting inside parent
+/// operations (`_r<num>`) and that operation's region (`_s<num>`). The region
+/// number is omitted if the parent operation has just one region. If a loop
+/// nest just consists of canonical loops nested inside each other, also uses
+/// `d<num>` where <num> is the nesting depth of the loop.
----------------
Meinersbur wrote:

Made the comment more detailed. It was correct albeit confusing; "nesting inside parent operations" meant the region ("r") out of an operation's list of region arguments, which is some form of nesting.

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