[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