[Mlir-commits] [mlir] [MLIR][SCF] Add checks to verify that the pipeliner schedule is correct. (PR #77083)
    Rik Huijzer 
    llvmlistbot at llvm.org
       
    Mon Jan  8 13:41:55 PST 2024
    
    
  
================
@@ -156,6 +160,11 @@ bool LoopPipelinerInternal::initializeLoopInfo(
     }
   }
 
+  if (!verifySchedule()) {
+    LDBG("--invalid schedule: " << op << " -> BAIL");
+    return false;
+  }
----------------
rikhuijzer wrote:
nit: should the implementation of `verifySchedule` not be defined above the function that contains this check (above `LoopPipelinerInternal`?) Most of the MLIR codebase and this file `LoopPipelining.cpp` do this.
https://github.com/llvm/llvm-project/pull/77083
    
    
More information about the Mlir-commits
mailing list