On 28/08/2025 14:19, Robin Dapp wrote:
LGTM. Are the IEEE-min/max variants in your plans as well?
I haven't checked if we propagate into those at all (unspec), though.
Yes, the IEEE variants are coming up next.
Thanks,
--
PA
On 28/08/2025 17:23, Kito Cheng wrote:
+ [(set_attr "type" "vfmuladd")]
this seems incorrect, otherwise LGTM
Good catch! It should be "vfminmax" -- copy-paste mistake.
Thanks,
--
PA
Paul-Antoine Arras 於 2025年8月28日 週四,01:50寫道:
> This pattern enables the combine pass (or late-combine, depending on the
> case)
> to merge a vec_duplicate into an smin RTL instruction.
>
> Before this patch, we have two instructions, e.g.:
> vfmv.v.f v2,fa0
> vfmin.vv v1,v1,v2
>
> A
LGTM. Are the IEEE-min/max variants in your plans as well?
I haven't checked if we propagate into those at all (unspec), though.
--
Regards
Robin
This pattern enables the combine pass (or late-combine, depending on the case)
to merge a vec_duplicate into an smin RTL instruction.
Before this patch, we have two instructions, e.g.:
vfmv.v.f v2,fa0
vfmin.vv v1,v1,v2
After, we get only one:
vfmin.vf v1,v1,fa0
gcc/Change