[llvm] [Reassociate] Use uint64_t for repeat count (PR #94232)

Yingwei Zheng via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 3 08:18:15 PDT 2024


================
@@ -625,10 +532,7 @@ static bool LinearizeExprTree(Instruction *I,
       // Node initially thought to be a leaf wasn't.
       continue;
     assert(!isReassociableOp(V, Opcode) && "Shouldn't be a leaf!");
-    APInt Weight = It->second;
-    if (Weight.isMinValue())
-      // Leaf already output or weight reduction eliminated it.
-      continue;
+    uint64_t Weight = It->second;
----------------
dtcxzyw wrote:

As we don't reduce weight for nilpotent ops now (e.g., xor), the weight cannot be zero.


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


More information about the llvm-commits mailing list