[llvm] [Reassociate] Don't reassociate vXi1 logical expressions (PR #123329)

Nikita Popov via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 20 07:46:50 PST 2025


================
@@ -2180,7 +2180,7 @@ void ReassociatePass::OptimizeInst(Instruction *I) {
   // is not further optimized, it is likely to be transformed back to a
   // short-circuited form for code gen, and the source order may have been
   // optimized for the most likely conditions.
-  if (I->getType()->isIntegerTy(1))
+  if (I->getType()->isIntOrIntVectorTy(1))
----------------
nikic wrote:

With the extension to vector types, this comment needs an update. For vector types there's not going to be any short-circuiting. 

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


More information about the llvm-commits mailing list