[PATCH] D94641: [AArch64][GlobalISel] Use wzr/xzr for FP stores of zero
Jessica Paquette via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jan 14 11:36:36 PST 2021
paquette added a comment.
In D94641#2498944 <https://reviews.llvm.org/D94641#2498944>, @aemerson wrote:
> It it possible to extend the existing combine in prelegalizer combiner to handle this too? Or will that cause us to still miss some cases?
The problem with that combine is it looks for G_FCONSTANTS which are only used by stores, so as is, it would miss cases.
I think that we could probably replace that combine with something similar to this though.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D94641/new/
https://reviews.llvm.org/D94641
More information about the llvm-commits
mailing list