[PATCH] D61695: [InstCombine] When turning sext into zext due to known bits, return the new ZExt instead of calling replaceinstuseswith

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 8 13:43:56 PDT 2019


spatel accepted this revision.
spatel added a comment.
This revision is now accepted and ready to land.

LGTM - there are several other transforms scattered around instcombine that produce a value with the builder and then manually replace.

These are almost NFC changes, but you can show the cosmetic diffs in regression tests by re-running 'update_test_checks.py'.


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

https://reviews.llvm.org/D61695





More information about the llvm-commits mailing list