[PATCH] D120953: [AArch64][SelectionDAG] Supports unpklo/hi instructions to reduce the number of loads

Allen zhong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Mar 12 08:07:40 PST 2022


Allen added a comment.

In D120953#3374916 <https://reviews.llvm.org/D120953#3374916>, @paulwalker-arm wrote:

> By directly supported I mean those which are legal and thus have isel patterns. Which I think boils down to:
>
>   

Thanks very much, this version I firstly fix the crash as you mention, and this version only play as the beginning of such optimisation.

If you agree, then I'll refact with your advice on the next version (**it is safest to exclude all scalable vector extending loads/truncating stores
and then selectively enable those which we directly support.** ), and add more cases to guard them.


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

https://reviews.llvm.org/D120953



More information about the llvm-commits mailing list