[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] AVX support for TCG
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] AVX support for TCG |
Date: |
Fri, 28 Dec 2018 14:13:13 +0000 |
On Fri, 28 Dec 2018 at 13:45, Nick Renieris <address@hidden> wrote:
> Also, I hope you meant four months for me, not for you - I'm
> completely new to the QEMU codebase. I expect it will take me weeks
> just to understand x86's 'translate.c' (who thought it'd be a good
> idea to put all this stuff in _one_ file?).
x86 suffers from being one of the first and oldest frontends,
and on top of that from not having had a great deal of active
attention. So it has a lot of remnants from older styles of
implementation, as well as the kind of "one big function in
one huge file" that organic growth of a codebase tends to give you.
It doesn't make that much difference whether you have one file
or several, though -- target/arm/translate-a64.c is pretty
well structured and easy (IMHO) to comprehend, but it's
5000 lines longer than target/i386/translate.c. The comprehensibility
improvements come from better breakdown into separate functions
and much clearer commenting of the extent of the decode at any
particular point (plus not having any legacy baggage to deal with).
thanks
-- PMM
- [Qemu-devel] AVX support for TCG, Nick Renieris, 2018/12/25
- Re: [Qemu-devel] AVX support for TCG, Richard Henderson, 2018/12/25
- Re: [Qemu-devel] AVX support for TCG, Nick Renieris, 2018/12/25
- Re: [Qemu-devel] AVX support for TCG, Richard Henderson, 2018/12/25
- Re: [Qemu-devel] AVX support for TCG, Nick Renieris, 2018/12/28
- Re: [Qemu-devel] AVX support for TCG,
Peter Maydell <=
- Re: [Qemu-devel] AVX support for TCG, Nick Renieris, 2018/12/28
- Re: [Qemu-devel] AVX support for TCG, Peter Maydell, 2018/12/28
- Re: [Qemu-devel] AVX support for TCG, Nick Renieris, 2018/12/28
- Re: [Qemu-devel] AVX support for TCG, Alex Bennée, 2018/12/28
- Re: [Qemu-devel] AVX support for TCG, Richard Henderson, 2018/12/29
- Re: [Qemu-devel] AVX support for TCG, Nick Renieris, 2018/12/30
- Re: [Qemu-devel] AVX support for TCG, Richard Henderson, 2018/12/30