[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] tcg: Add doxygen documentation to the tcg front
From: |
Lluís Vilanova |
Subject: |
Re: [Qemu-devel] [PATCH] tcg: Add doxygen documentation to the tcg frontend |
Date: |
Wed, 14 Jan 2015 19:15:09 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.4 (gnu/linux) |
Bastian Koppelmann writes:
> On 01/14/2015 03:36 PM, Lluís Vilanova wrote:
>> Bastian Koppelmann writes:
>>
>>> There is no overview, that shows all the frontend operation one can use, as
>>> found on the wiki. Unfortunatly the wiki is out of date, so let's try to
>>> move
>>> this documentation to the source files, which has the benefit, that it is
>>> easy
>>> to update the documentation, if the frontend is changed. This patch adds
>>> doxygen
>>> tags to all the 32 bit versions of the tcg frontend operations, because the
>>> 64 bit version would mostly have the same documentation, and all the type
>>> conversition operations. The file tag has a note, that makes the user aware
>>> of
>>> the missing 64 operations. In this version all the immediate variants are
>>> also
>>> documented by simply refering to the non immediate version. However I'm
>>> willing
>>> to drop that.
>>> Any comments?
>> The operations (or at least most of them) are already documented in
>> tcg/README. If this change is accepted, I'd rather move the contents of the
>> README file into here.
> The README file contains IIRC only operation, which are also represented by
> the
> intermediate format of tcg. Unfortunately this excludes useful helpers like
> subfi. So Richard suggested commenting the header file. However I wouldn't add
> only those helpers like subfi to the doxygen documentation, since the idea is
> to
> have a good overview. If you have another suggestion I'm happy to hear it :).
Sorry I wasn't clear. I meant that it might be better to remove tcg/README and
instead document all the operations (those in the readme and the additional
ones) in the header.
Thanks,
Lluis
--
"And it's much the same thing with knowledge, for whenever you learn
something new, the whole world becomes that much richer."
-- The Princess of Pure Reason, as told by Norton Juster in The Phantom
Tollbooth
Re: [Qemu-devel] [PATCH] tcg: Add doxygen documentation to the tcg frontend, Bastian Koppelmann, 2015/01/14
Re: [Qemu-devel] [PATCH] tcg: Add doxygen documentation to the tcg frontend, Bastian Koppelmann, 2015/01/21