One more piece of target/arm prep work from my SVE branch.

I saw that Alex was touching the same bit of code in his ARMv8.2
fp16 patch set and thought we should coordinate on this.


r~


Richard Henderson (2):
  target/arm: Split out vfp_expand_imm
  target/arm: Add fp16 support to vfp_expand_imm

 target/arm/translate-a64.c | 49 +++++++++++++++++++++++++++++++---------------
 1 file changed, 33 insertions(+), 16 deletions(-)

-- 
2.13.6

Richard Henderson <richard.henderson@linaro.org> writes:

> One more piece of target/arm prep work from my SVE branch.
>
> I saw that Alex was touching the same bit of code in his ARMv8.2
> fp16 patch set and thought we should coordinate on this.


Thanks, I'll roll them into my series.

>
>
> r~
>
>
> Richard Henderson (2):
>   target/arm: Split out vfp_expand_imm
>   target/arm: Add fp16 support to vfp_expand_imm
>
>  target/arm/translate-a64.c | 49 +++++++++++++++++++++++++++++++---------------
>  1 file changed, 33 insertions(+), 16 deletions(-)


--
Alex Bennée

On 10 January 2018 at 06:33, Richard Henderson
<richard.henderson@linaro.org> wrote:
> One more piece of target/arm prep work from my SVE branch.
>
> I saw that Alex was touching the same bit of code in his ARMv8.2
> fp16 patch set and thought we should coordinate on this.
>
>
> r~
>
>
> Richard Henderson (2):
>   target/arm: Split out vfp_expand_imm
>   target/arm: Add fp16 support to vfp_expand_imm
>
>  target/arm/translate-a64.c | 49 +++++++++++++++++++++++++++++++---------------
>  1 file changed, 33 insertions(+), 16 deletions(-)

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>

I could put these in target-arm.next now if you like ?

thanks
-- PMM

On 01/12/2018 09:22 AM, Peter Maydell wrote:
>> Richard Henderson (2):
>>   target/arm: Split out vfp_expand_imm
>>   target/arm: Add fp16 support to vfp_expand_imm
>>
>>  target/arm/translate-a64.c | 49 +++++++++++++++++++++++++++++++---------------
>>  1 file changed, 33 insertions(+), 16 deletions(-)
> 
> Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
> 
> I could put these in target-arm.next now if you like ?

Yes, please.


r~

On 12 January 2018 at 19:07, Richard Henderson
<richard.henderson@linaro.org> wrote:
> On 01/12/2018 09:22 AM, Peter Maydell wrote:
>>> Richard Henderson (2):
>>>   target/arm: Split out vfp_expand_imm
>>>   target/arm: Add fp16 support to vfp_expand_imm
>>>
>>>  target/arm/translate-a64.c | 49 +++++++++++++++++++++++++++++++---------------
>>>  1 file changed, 33 insertions(+), 16 deletions(-)
>>
>> Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
>>
>> I could put these in target-arm.next now if you like ?
>
> Yes, please.

OK, done.

thanks
-- PMM