[PATCH] D148123: [AArch64][CostModel] Make sext/zext free if folded into a masked load
    David Sherwood via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Wed Apr 19 05:59:34 PDT 2023
    
    
  
david-arm added a comment.
In D148123#4277223 <https://reviews.llvm.org/D148123#4277223>, @dmgreen wrote:
> That would make sense for normal loads, but masked loads will not split like that (unless they can extend the mask). https://godbolt.org/z/x9T8vP6Kx. It may be simpler to be more precise about the cost if it returned it directly.
Hi @dmgreen , 've rebased this patch now that https://reviews.llvm.org/D142456 has landed, which explicitly accounts for extends to illegal types, so I think your concerns should be addressed now!
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D148123/new/
https://reviews.llvm.org/D148123
    
    
More information about the llvm-commits
mailing list