> Evandro Menezes wrote: > Hi, Wilco. > > It looks good to me, but FCMP is quite different from FCCMP on Exynos M1, > so it'd be helpful to have distinct types for them. Say, "fcmp{s,d}" > and "fccmp{s,d}". Would it be acceptable to add this with this patch or > later?
It would be easy to add fccmps/d as new attributes, I prefer to do that as a separate patch. Are there any other attributes that you think are missing besides the ones I know about (extr, 64-bit mul/mla, mulh, bfi)? Also would we need a new entry in the cost tables? Wilco