[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