[PATCH] D79041: [InstCombine] Fold or(zext(bswap(x)),shl(zext(bswap(y)),bw/2)) -> bswap(or(zext(x),shl(zext(y), bw/2))
Roman Lebedev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri May 1 23:57:37 PDT 2020
lebedev.ri accepted this revision.
lebedev.ri added a comment.
This revision is now accepted and ready to land.
This looks good to me, but i'm not sure this will work more generally.
I.e., i will be surprized if the `or` tree reaches us in the proper form,
without reassociation into some other form we aren't expecting.
Especially for larger tree depths.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D79041/new/
https://reviews.llvm.org/D79041
More information about the llvm-commits
mailing list