================
@@ -1425,8 +1453,14 @@ static bool runImpl(Module &M, AnalysisGetter &AG, 
TargetMachine &TM,
     }
   }
 
-  ChangeStatus Change = A.run();
-  return Change == ChangeStatus::CHANGED;
+  bool Changed = A.run() == ChangeStatus::CHANGED;
+
+  if (Changed && (LTOPhase == ThinOrFullLTOPhase::None ||
+                  LTOPhase == ThinOrFullLTOPhase::FullLTOPostLink ||
+                  LTOPhase == ThinOrFullLTOPhase::ThinLTOPostLink))
+    checkWavesPerEU(M, TM);
----------------
arsenm wrote:

This isn't running as part of the attribute run? Why isn't this part of 
manifest? 

Changed is also really course, you only can care about specific attributes 
changing 

https://github.com/llvm/llvm-project/pull/123995
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to