[PATCH] D152281: [Transforms][LICM] Add the ability to undo unprofitable reassociation
Paul Osmialowski via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jul 28 15:37:38 PDT 2023
pawosm01 marked an inline comment as done.
pawosm01 added inline comments.
================
Comment at: llvm/lib/Transforms/Scalar/LICM.cpp:2737
+ Instruction *Ins = cast<Instruction>(U->getUser());
+ IRBuilder<> Builder(Preheader->getTerminator());
+ U->set(Builder.CreateFMulFMF(U->get(), Factor, Ins, "factor.op.fmul"));
----------------
nikic wrote:
> nit: Can move this IRBuilder outside the loop.
Ah, yeah, good catch; corrected.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D152281/new/
https://reviews.llvm.org/D152281
More information about the llvm-commits
mailing list