Re: [PATCH 1/2] msvcrt: Implement fesetenv for aarch64

[Martin Storsjö]

On Tue, 3 Aug 2021, Piotr Caban wrote:
> Hi Martin,
>
> On 8/1/21 11:04 PM, Martin Storsjo wrote:
> > +#elif defined(__aarch64__)
> > - _control87(env->_Fe_ctl, 0xffffffff);
>
> Please use fenv_decode helper so it can also work in ucrtbase where fegetenv 
> uses fenv_encode.

Oh, right, I missed how that aspect worked - will do.
> > - ULONG_PTR fpsr;
>
> Please define new variables at the start of a block.

Oh, oops, will do.

// Martin