[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