[PATCH] D152281: [Transforms][LICM] Add the ability to undo unprofitable reassociation

[Quentin Colombet via Phabricator via llvm-commits]

qcolombet added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LICM.cpp:2677
 
+static bool hoistFPAssociation(Instruction &I, Loop &L,
+                               ICFLoopSafetyInfo &SafetyInfo,
----------------
Could you add a comment on what this function is doing and what it means to return true or false?

It'll make the review easier.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D152281/new/

https://reviews.llvm.org/D152281