On Sat, Nov 14, 2015 at 1:36 AM, Wilco Dijkstra <wilco.dijks...@arm.com> wrote: >> 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?
For ThunderX they can treated as the same. Thanks, Andrew > > Wilco > >