Hello,

On Wed, 23 Nov 2022, Mark Cave-Ayland wrote:
This is something I've had lying around in the background for a while, and I
finally managed to get it in a state suitable for submission. The main
motivation was that I wanted to have a TCG operation reference that was
easily viewable online in a web browser.

Nice. Maybe including a link to the result could help review. Is this the right link?

https://github.com/mcayland/qemu/blob/tcg-readme-rst/docs/devel/tcg.rst

The second paragraph refers to tcg/README which is this file and this patch moves it so that sentence does not make much sense. Maybe it should be dropped and the only remaining sentence from that paragraph added to the previous. In fact the last sentence could also be added to avoid one sentence paragraphs and make the intro section just one paragraph.

Regards,
BALATON Zoltan

The patch converts tcg/README to rst and then moves it from tcg/README to a
new "TCG Intermediate Representation" section in docs/devel.

The conversion is fairly straightforward, with a few minor touch-ups to
produce a more aesthetic result which are documented in the commit message.

Signed-off-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk>


Mark Cave-Ayland (1):
 tcg: convert tcg/README to rst

docs/devel/atomics.rst   |    2 +
docs/devel/index-tcg.rst |    1 +
docs/devel/tcg-ops.rst   | 1018 ++++++++++++++++++++++++++++++++++++++
tcg/README               |  784 -----------------------------
4 files changed, 1021 insertions(+), 784 deletions(-)
create mode 100644 docs/devel/tcg-ops.rst
delete mode 100644 tcg/README



Reply via email to