Message ID | 20180110063337.21538-1-richard.henderson@linaro.org |
---|---|
Headers | show |
Series | target/arm: split out vfp_expand_imm | expand |
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