[PATCH] D105853: [ISel] Expand saddsat and ssubsat via asr and xor

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 18 13:30:46 PDT 2021


RKSimon added a comment.

In D105853#2952726 <https://reviews.llvm.org/D105853#2952726>, @dmgreen wrote:

> In D105853#2952718 <https://reviews.llvm.org/D105853#2952718>, @RKSimon wrote:
>
>> The X86 vector changes look OK to me - we've reduced usage of variable blends (blendvps/blendvpd) which tends to be a particular bottleneck.
>
> OK thanks. Let me know whether to remove the v2i64 case or not.

Your v2i64 change is fine.


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

https://reviews.llvm.org/D105853



More information about the llvm-commits mailing list