mbox series

[0/2] target/arm: split out vfp_expand_imm

Message ID 20180110063337.21538-1-richard.henderson@linaro.org
Headers show
Series target/arm: split out vfp_expand_imm | expand

Message

Richard Henderson Jan. 10, 2018, 6:33 a.m. UTC 
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

Comments

Alex Bennée Jan. 10, 2018, 10:11 a.m. UTC | #1 
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
Peter Maydell Jan. 12, 2018, 5:22 p.m. UTC | #2 
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
Richard Henderson Jan. 12, 2018, 7:07 p.m. UTC | #3 
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~
Peter Maydell Jan. 15, 2018, 1:13 p.m. UTC | #4 
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