[PATCH] D122166: [InstCombine] try to narrow shifted bswap-of-zext

Wed Mar 23 03:55:52 PDT 2022

spatel added a comment.

In D122166#3401171 <https://reviews.llvm.org/D122166#3401171>, @nathanchance wrote:

> I have reverted this in 4e0008dcbe9fce99b9727e8bbeb129efc7bf2d80 <https://reviews.llvm.org/rG4e0008dcbe9fce99b9727e8bbeb129efc7bf2d80> to hopefully keep our CI from going completely red. I am happy to test a new revision as needed.

Thanks for the revert! I missed a check on the source type size, so we tried to create a bogus bswap of i24. I'll add a regression test based on the provided source file and update the patch.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122166