[llvm] [DFAJumpThreading] Set MadeChanges only if threading happened (PR #162241)

Nikita Popov via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 7 01:50:31 PDT 2025


================
@@ -820,11 +820,14 @@ struct TransformDFA {
       : SwitchPaths(SwitchPaths), DT(DT), AC(AC), TTI(TTI), ORE(ORE),
         EphValues(EphValues) {}
 
-  void run() {
+  bool run() {
+    bool MadeChange = false;
     if (isLegalAndProfitableToTransform()) {
       createAllExitPaths();
+      MadeChange = true;
       NumTransforms++;
----------------
nikic wrote:

nit: You could just return true here and return false below. No need for the variable.

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


More information about the llvm-commits mailing list