[PATCH] D124183: [InstCombine] Add one use limitation for (X * C2) << C1 --> X * (C2 << C1)

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a comment.

We may be going in circles here.
Both of these transforms do not require one-use check, neither from correctness perspective,
nor from the profitability perspective (at least, as instcombine views it).
I'm not sure it will be productive if i simply repeat everything i have already stated,
so i'm not sure how to approach the disagreement differently.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124183