[PATCH] D90382: [InstCombine] foldSelectRotate - generalize to foldSelectFunnelShift

[Nikita Popov via Phabricator via llvm-commits]

nikic added a comment.

@nlopes I think we should adjust the funnel shift definition to say that it blocks poison on one operand if the shift amount is zero. Basically the poison semantics should be "as if" the funnel shift were expanded, which does include an explicit select for the zero shift amount case.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90382