[Diffusion] rL341831: InstCombine: move hasOneUse check to the top of foldICmpAddConstant

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a comment.

(resubmitting with lists added)

@t.p.northover why was this change done? There is no real motivation stated in the commit message. Can this be reverted?
The old position of one-use check was correct, it guarded only the transforms that had to be guarded - the ones that produced two instructions.


https://reviews.llvm.org/rL341831