Currently InstCombine folds using the return replaceInstUsesWith(V, Builder.CreateFoo()) pattern do not preserve the original name of the instruction. To preserve the name, you either have to use something like return FooInst::Create(...) which is usually less nice, or go out of the way to preserve the name with takeName(). We often don't do that.
This patch instead preserves the name in replaceInstUsesWith() when replacing a named instruction with an unnamed instruction. To be conservative, I also added a zero-use check, which is a proxy for the case where the instruction was just created, rather than an existing one reused. Possibly we could drop that part.
As InstCombine tests are robust against renames this does not cause any test diffs, so I regenerated a random test to show the effects.