> The Neon versions of int-float conversions need their own helper routines > because they must use the "standard FPSCR" rather than the default one. > Refactor the helper functions to make it easy to add the neon versions. > While we're touching the code, move the helpers to op_helper.c so that > we can use the global env variable rather than passing it as a parameter.
IMO this is going in the wrong direction. We should in aiming for less implicit accesses to cpu state, not more. Maybe better would be to explicitly pass a pointer the fp status. That way you don't even need separate VFP and NEON variants of these routines. Paul