[PATCH] D93818: [LangRef] Update shufflevector's semantics to return poison if the mask is undef

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 27 12:38:48 PST 2020


nikic added a comment.

Using `undef` in the shuffle mask to indicate a `poison` result seems pretty confusing to me. I think we should use `poison` as the marker instead. We could allow both undef/poison in input IR, but at least the output IR rendering should use poison (the internal representation is something else anyway, as shuffle masks are no longer Values).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93818



More information about the llvm-commits mailing list