[PATCH] D57348: [CodeGen][X86] Don't scalarize vector saturating add/sub

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 8 10:57:47 PDT 2019


nikic added a comment.

@aemerson Thanks for the report! From a quick look, looks like the usubsat expands to usubo, which is then expanded during op legalization, while it needs to be expanded during vector op legalization. We're doing that for umulo and smulo, but apparently not for uaddo/saddo and usubo/ssubo. I'll prepare a patch.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D57348





More information about the llvm-commits mailing list