[llvm] [llvm][CodeGen] Added a new restriction for II by pragma in window scheduler (PR #99448)

Kai Yan via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 22 01:29:10 PDT 2024


================
@@ -528,8 +528,16 @@ bool MachinePipeliner::useSwingModuloScheduler() {
 }
 
 bool MachinePipeliner::useWindowScheduler(bool Changed) {
-  // WindowScheduler does not work when it is off or when SwingModuloScheduler
-  // is successfully scheduled.
+  // WindowScheduler does not work for following cases:
+  // 1. when it is off.
+  // 2. when SwingModuloScheduler is successfully scheduled.
+  // 3. when pragma II is enabled.
+  if (II_setByPragma) {
+    LLVM_DEBUG(
+        dbgs() << "Window scheduling is disabled when Pragma II is set.\n");
----------------
kaiyan96 wrote:

Is it better to change the debug message to:
```
  if (II_setByPragma) {
    LLVM_DEBUG(
        dbgs() << "Window scheduling is disabled when llvm.loop.pipeline.initiationinterval is set.\n");
  }
```

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


More information about the llvm-commits mailing list