On Thu, Nov 6, 2025 at 8:56 AM Karl Meakin via Sourceware Forge <[email protected]> wrote: > > > From: Karl Meakin <[email protected]> > > The bodies of `mov<ALLI>cc` and `mov<GPF>cc` are identical, so merge > them by using a new mode iterator that combines `ALLI` and `GPF`.
This patch is a good improvement, ok. Note a related issue is fcsel is not produced for BF or HF modes even though you could. See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114829 for a good example. Without full `fscel h` support, using s sized register should be ok. Thanks, Andrew > > gcc/ChangeLog: > > * config/aarch64/aarch64.md (mov<ALLI>cc): Merge with ... > (mov<ALLI>cc): ... this. > * config/aarch64/iterators.md(ALLI_GPF): New mode iterator. > --- > gcc/config/aarch64/aarch64.md | 35 +++++---------------------------- > gcc/config/aarch64/iterators.md | 4 ++++ > 2 files changed, 9 insertions(+), 30 deletions(-) >
