Re: [PATCH v8 00/35] Hexagon patch series

[Richard Henderson]

On 2/7/21 9:45 PM, Taylor Simpson wrote:
> This series adds support for the Hexagon processor with Linux user support
> 
> See patch 02 Hexagon README for detailed information.
> 
> This series assumes int128_or() is implemented.
> https://lists.nongnu.org/archive/html/qemu-devel/2020-10/msg06004.html
> 
> The series is also available at https://github.com/quic/qemu on branch
> small_series_v8.
> 
> 
> Once the series is applied, the Hexagon port will pass "make check-tcg".
> The series also includes Hexagon-specific tests in tcg/tests/hexagon.
> 
> The final patch in the series add docker support.  Thanks to Alessandro
> Di Federico <ale@rev.ng> and Brian Cain <bcain@quicinc.com> for making this
> happen.  The default container (debian-hexagon-cross) uses a toolchain built
> by rev.ng.  Alternatively, there is a container that will build the toolchain
> locally (debian-hexagon-cross-build-local).

Right.  This is in really good shape.

I've completed review of this round, and there are some nits.  But they're
minor enough that I wouldn't even mind them being addressed via the normal
development process.  I.e. I'd be keen to not look through that diffstat again.
 ;-)

Any objections from anyone else on that?

I don't suppose you and Peter Maydell signed gpg keys when we all met in Lyon?


r~