[llvm] 8b167db - [CodeGen] Fix a warning

Kazu Hirata via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 5 01:18:40 PDT 2025


Author: Kazu Hirata
Date: 2025-06-05T01:18:34-07:00
New Revision: 8b167db63a9d86fb1c558925c3d76d782d78b034

URL: https://github.com/llvm/llvm-project/commit/8b167db63a9d86fb1c558925c3d76d782d78b034
DIFF: https://github.com/llvm/llvm-project/commit/8b167db63a9d86fb1c558925c3d76d782d78b034.diff

LOG: [CodeGen] Fix a warning

This patch fixes:

  llvm/lib/CodeGen/MacroFusion.cpp:65:12: error: unused variable
  'FirstCluster' [-Werror,-Wunused-variable]

  llvm/lib/CodeGen/MacroFusion.cpp:66:12: error: unused variable
  'SecondCluster' [-Werror,-Wunused-variable]

Added: 
    

Modified: 
    llvm/lib/CodeGen/MacroFusion.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/CodeGen/MacroFusion.cpp b/llvm/lib/CodeGen/MacroFusion.cpp
index 71053342f468e..975a3fe71abad 100644
--- a/llvm/lib/CodeGen/MacroFusion.cpp
+++ b/llvm/lib/CodeGen/MacroFusion.cpp
@@ -62,9 +62,8 @@ bool llvm::fuseInstructionPair(ScheduleDAGInstrs &DAG, SUnit &FirstSU,
     if (SI.isCluster())
       return false;
 
-  unsigned FirstCluster = FirstSU.ParentClusterIdx;
-  unsigned SecondCluster = SecondSU.ParentClusterIdx;
-  assert(FirstCluster == InvalidClusterId && SecondCluster == InvalidClusterId);
+  assert(FirstSU.ParentClusterIdx == InvalidClusterId &&
+         SecondSU.ParentClusterIdx == InvalidClusterId);
 
   // Though the reachability checks above could be made more generic,
   // perhaps as part of ScheduleDAGInstrs::addEdge(), since such edges are valid,


        


More information about the llvm-commits mailing list