[InstCombine] When turning sext into zext due to known bits, return the new ZExt...
authorCraig Topper <craig.topper@intel.com>
Wed, 8 May 2019 20:59:21 +0000 (20:59 +0000)
committerCraig Topper <craig.topper@intel.com>
Wed, 8 May 2019 20:59:21 +0000 (20:59 +0000)
commit51a17df45d198731ca9dc691ef90b38c4d0810ac
tree29b4fc3dedac11a7ae6557323a408e845bd15aa7
parent73b7272e7a87be08e4f5775dbd163685d57558a1
[InstCombine] When turning sext into zext due to known bits, return the new ZExt instead of calling replaceinstuseswith

The worklist loop that we're returning back to should be able to do the repacement itself. This is how we normally do replacements.

My main motivation was that I observed that we weren't preserving the name of the result when we do this transform. The replacement code in the worklist loop will call takeName as part of the replacement.

Differential Revision: https://reviews.llvm.org/D61695

llvm-svn: 360284
llvm/lib/Transforms/InstCombine/InstCombineCasts.cpp