[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#41615: [feature/native-comp] Dump prettier C code.
From: |
Andrea Corallo |
Subject: |
bug#41615: [feature/native-comp] Dump prettier C code. |
Date: |
Sun, 31 May 2020 11:37:50 +0000 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux) |
Nicolas Bértolo <nicolasbertolo@gmail.com> writes:
> I have reformatted the patches.
>
> Sorry for the inconveniences.
>
>
> From 0720ec7eb3dc552b018273cd68a5f7d6bb2fdb72 Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?Nicol=C3=A1s=20B=C3=A9rtolo?= <nicolasbertolo@gmail.com>
> Date: Wed, 20 May 2020 00:34:32 -0300
> Subject: [PATCH 1/2] Define static data using string literals.
>
> The purpose of this change is to dump prettier C files.
> This does not affect compilation times in my tests.
>
> * src/comp.c (emit_static_object): Define static objects using string
> literals, memcpy and bzero.
> ---
> src/comp.c | 295 +++++++++++++++++++++++++++++++++++++++++++++++++----
> 1 file changed, 277 insertions(+), 18 deletions(-)
I like this considerably less :)
It introduces quite some complexity and the same advantage in
debuggability can be achieved with something like the attached 8 line
patch (untested).
Generally speaking I want to try to keep our back-end as simple as we
manage to.
On the subject of 'emit_static_object' the current situation is not
ideal. But rather that working around the workaround I believe the right
thing to do is to improve GCC with a new entry point and keep the
current arrangement as a simple fallback.
I've already an half cooked GCC patch to allow for directly injecting
blobs, this should have more then one advantage. Hopefully I manage to
start testing it today, I'm rather curious.
Andrea
--
akrl@sdf.org
0001-Emit-better-debug-comments-in-emit_static_object.patch
Description: Text Data
- bug#41615: [feature/native-comp] Dump prettier C code., Nicolas Bértolo, 2020/05/30
- bug#41615: [feature/native-comp] Dump prettier C code., Nicolas Bértolo, 2020/05/30
- bug#41615: [feature/native-comp] Dump prettier C code., Andrea Corallo, 2020/05/31
- bug#41615: [feature/native-comp] Dump prettier C code.,
Andrea Corallo <=
- bug#41615: [feature/native-comp] Dump prettier C code., Andrea Corallo, 2020/05/31
- bug#41615: [feature/native-comp] Dump prettier C code., Nicolas Bértolo, 2020/05/31
- bug#41615: [feature/native-comp] Dump prettier C code., Andrea Corallo, 2020/05/31
- bug#41615: [feature/native-comp] Dump prettier C code., Nicolas Bértolo, 2020/05/31
- bug#41615: [feature/native-comp] Dump prettier C code., Andrea Corallo, 2020/05/31
- bug#41615: [feature/native-comp] Dump prettier C code., Nicolas Bértolo, 2020/05/31
- bug#41615: [feature/native-comp] Dump prettier C code., Andrea Corallo, 2020/05/31
- bug#41615: [feature/native-comp] Dump prettier C code., Andrea Corallo, 2020/05/31