[PATCH] D127680: [BasicTTI] Allow generic handling of scalable vector fshr/fshl

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 14 14:11:39 PDT 2022


reames added a comment.

In D127680#3581448 <https://reviews.llvm.org/D127680#3581448>, @david-arm wrote:

> I suspect we were just being overly conservative at the time when fixing up some FixedVectorType casts, but as you say it seems to work now. Would you be able to add a similar Analysis/CostModel/AArch64/sve-intrinsics.ll test as well for the <vscale x 4 x i32> case?

@david-arm It's not really clear which test you want me to add.  Since this isn't fixing a crash, could you just land what you have in mind and I'll rebase this over?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127680



More information about the llvm-commits mailing list