[llvm] [LICM][WIP] Make an integer version of hoistFPAssociation. (PR #67736)

Philip Reames via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 2 12:32:30 PDT 2023


================
@@ -2719,6 +2728,65 @@ static bool hoistFPAssociation(Instruction &I, Loop &L,
   return true;
 }
 
+static bool hoistIntAssociation(Instruction &I, Loop &L,
----------------
preames wrote:

Comment as much on the existing code as the new code.  We're going through the following steps:
0:  ((A1 * B1) + (A2 * B2) + ...) * C where
1: ((A1 * C * B1) + (A2 * C * B2) + ...) 
2: ((A1 * C) * B1 + (A2 * C) * B2 + ...) 

Isn't the step going from 0 to 1 distribution, not associativity?  If so, some comments need clarified.  


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


More information about the llvm-commits mailing list