On 9 September 2012 13:25, Max Filippov <jcmvb...@gmail.com> wrote: > On Sun, Sep 9, 2012 at 2:05 PM, Peter Maydell <peter.mayd...@linaro.org> > wrote: >> Given that these are just 'v &= 0x7fffffff' and 'v ^= 0x80000000' >> it seems like it would be better to just generate code for them >> rather than calling out to a helper, though in some ways it does >> break the abstraction layer of the softfloat library. I've been >> toying with the idea of doing this for target-arm. > > I doubt that these opcodes are used often enough to justify such > hack. > >> (we could have gen_float32_abs() and gen_float32_chs() in >> softfloat.h, or would that be a worse layering violation in >> the other direction? I dunno.) > > IMHO this approach is a bit cleaner.
Mmm, maybe. I think you can reasonably leave your patch as it is, anyway. -- PMM