[PATCH] D151616: [Transforms][Reassociate] "Reassociate expressions" pass optimizations not always profitable

[Florian Hahn via Phabricator via llvm-commits]

fhahn added a comment.

There's been some work to reassoicate in LICM to enable hoisting: D148001 <https://reviews.llvm.org/D148001>. It would probably be more general to extend the logic in LICM to handle those cases, which would also trigger if the user wrote the expression like `reassociate` creates at the moment.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151616