[PATCH] D141204: [GlobalISel] Extending G_SBFX/G_UBFX combine to consider G_SEXT_INREG as well

David Ferenc Szabo via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 12 06:02:36 PST 2023


DavidFerencSzabo added a comment.

In D141204#4035658 <https://reviews.llvm.org/D141204#4035658>, @foad wrote:

> This should be implemented by teaching computeNumSignBits about G_SBFX.

Thanks for the feedback. Upon inspection it turned out that AArch64 does not require that part of the change, hence I removed it.


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

https://reviews.llvm.org/D141204



More information about the llvm-commits mailing list