[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