[Openmp-commits] [clang] [llvm] [mlir] [openmp] [LoopTiling][Clang][OpenMP] Canonical Intra-tile Loops (PR #191114)

[Michael Kruse via Openmp-commits]

================
@@ -10,12 +10,48 @@
 //
 //===----------------------------------------------------------------------===//
 
-#include "clang/AST/ASTContext.h"
 #include "clang/AST/StmtOpenMP.h"
+#include "clang/AST/ASTContext.h"
+#include "clang/AST/Attr.h"
+#include "clang/AST/Stmt.h"
 
 using namespace clang;
 using namespace llvm::omp;
 
+// True if the attributed statement carries the internal `omp tile` intra-tile
+// body-guard marker.
+static bool hasOMPInvariantPredicateBound(const AttributedStmt *AS) {
+  for (const Attr *A : AS->getAttrs())
+    if (isa<OMPInvariantPredicateBoundAttr>(A))
+      return true;
+  return false;
+}
+
+// Like `Stmt::IgnoreContainers`, but also treats the `omp tile` body-guard
+static Stmt *IgnoreContainersAndOMPTileBodyGuard(Stmt *S) {
----------------
Meinersbur wrote:

```suggestion
static Stmt *ignoreContainersAndOMPTileBodyGuard(Stmt *S) {
```
Start functions with lower case letters: https://llvm.org/docs/CodingStandards.html#name-types-functions-variables-and-enumerators-properly

Not all functions have been renamed to follow the updated coding style, but for new functions use the current coding standard.

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