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

[Tom Eccles 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.
----------------
tblah wrote:

If I understand correctly
```suggestion
/// Generate a name of a canonical loop nest of the format
/// `<prefix>(_r<num>_(d|s)<num>)*` that describes its nesting inside parent
/// operations (`_s<num>`) and that operation's region (`_r<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.
```

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