[PATCH] D58877: [InstCombine] fold add(add(A, ~B), 1) -> sub(A, B)

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a comment.

The logic/tests look ok, but the question of necessity/usefulness has not been answered. Is there evidence that this pattern will ever reach instcombine in a normal optimization pipeline that includes -reassociate?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D58877