[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v2 0/5] misc: Enforce .[ch].inc extension for re-included .c/.h f
From: |
Philippe Mathieu-Daudé |
Subject: |
[PATCH v2 0/5] misc: Enforce .[ch].inc extension for re-included .c/.h files |
Date: |
Thu, 8 Jun 2023 15:31:03 +0200 |
Since v1:
- addressed Richard's easy review comments
Richard, this is what I had before looking at removing exec/helper*.
Posting in case it is useful enough before getting there.
--
Follow the convention to use the .inc extension for .c/.h files
re-included, as docummented in Coding Style since commit 6a0057aa22:
If you do use template header files they should be named with
the ``.c.inc`` or ``.h.inc`` suffix to make it clear they are
being included for expansion.
Most of the codebase already uses this extension:
$ git ls-files | fgrep .inc | wc -l
155
Rename some files left over.
Philippe Mathieu-Daudé (5):
hw/remote/proxy: Remove dubious 'event_notifier-posix.c' include
tests/tcg/aarch64: Rename bti-crt.inc.c -> bti-crt.c.inc
target/arm: Rename helper template headers as '.h.inc'
target/i386: Rename helper template headers as '.h.inc'
target: Bulk rename 'helper.h' -> 'helper.h.inc' for all targets
include/exec/helper-gen.h | 2 +-
include/exec/helper-proto.h | 2 +-
target/alpha/{helper.h => tcg/helper.h.inc} | 0
target/arm/tcg/{helper-a64.h => helper-a64.h.inc} | 0
target/arm/tcg/{helper-mve.h => helper-mve.h.inc} | 0
target/arm/tcg/{helper-sme.h => helper-sme.h.inc} | 0
target/arm/tcg/{helper-sve.h => helper-sve.h.inc} | 0
target/arm/{helper.h => tcg/helper.h.inc} | 8 ++++----
target/avr/{helper.h => tcg/helper.h.inc} | 0
target/cris/{helper.h => tcg/helper.h.inc} | 0
target/hexagon/{helper.h => tcg/helper.h.inc} | 0
target/hppa/{helper.h => tcg/helper.h.inc} | 0
.../{cc_helper_template.h => cc_helper_template.h.inc} | 0
target/i386/{helper.h => tcg/helper.h.inc} | 6 +++---
.../i386/{ops_sse_header.h => tcg/ops_sse_header.h.inc} | 0
.../shift_helper_template.h.inc} | 0
target/loongarch/{helper.h => tcg/helper.h.inc} | 0
target/m68k/{helper.h => tcg/helper.h.inc} | 0
target/microblaze/{helper.h => tcg/helper.h.inc} | 0
target/mips/{helper.h => tcg/helper.h.inc} | 0
target/nios2/{helper.h => tcg/helper.h.inc} | 0
target/openrisc/{helper.h => tcg/helper.h.inc} | 0
target/ppc/{helper.h => tcg/helper.h.inc} | 0
target/riscv/{helper.h => tcg/helper.h.inc} | 0
target/rx/{helper.h => tcg/helper.h.inc} | 0
target/s390x/{helper.h => tcg/helper.h.inc} | 0
target/sh4/{helper.h => tcg/helper.h.inc} | 0
target/sparc/{helper.h => tcg/helper.h.inc} | 0
target/tricore/{helper.h => tcg/helper.h.inc} | 0
target/xtensa/{helper.h => tcg/helper.h.inc} | 0
hw/remote/proxy.c | 1 -
target/alpha/translate.c | 2 +-
target/arm/tcg/translate.c | 2 +-
target/avr/translate.c | 2 +-
target/cris/translate.c | 2 +-
target/hexagon/translate.c | 2 +-
target/hppa/translate.c | 2 +-
target/i386/tcg/cc_helper.c | 8 ++++----
target/i386/tcg/int_helper.c | 8 ++++----
target/i386/tcg/translate.c | 2 +-
target/loongarch/translate.c | 2 +-
target/m68k/translate.c | 2 +-
target/microblaze/translate.c | 2 +-
target/mips/tcg/translate.c | 2 +-
target/nios2/translate.c | 2 +-
target/openrisc/translate.c | 2 +-
target/ppc/translate.c | 2 +-
target/riscv/translate.c | 2 +-
target/rx/translate.c | 2 +-
target/s390x/tcg/translate.c | 2 +-
target/sh4/translate.c | 2 +-
target/sparc/translate.c | 2 +-
target/tricore/translate.c | 2 +-
target/xtensa/translate.c | 2 +-
tests/tcg/aarch64/bti-1.c | 2 +-
tests/tcg/aarch64/bti-3.c | 2 +-
tests/tcg/aarch64/{bti-crt.inc.c => bti-crt.c.inc} | 0
57 files changed, 40 insertions(+), 41 deletions(-)
rename target/alpha/{helper.h => tcg/helper.h.inc} (100%)
rename target/arm/tcg/{helper-a64.h => helper-a64.h.inc} (100%)
rename target/arm/tcg/{helper-mve.h => helper-mve.h.inc} (100%)
rename target/arm/tcg/{helper-sme.h => helper-sme.h.inc} (100%)
rename target/arm/tcg/{helper-sve.h => helper-sve.h.inc} (100%)
rename target/arm/{helper.h => tcg/helper.h.inc} (99%)
rename target/avr/{helper.h => tcg/helper.h.inc} (100%)
rename target/cris/{helper.h => tcg/helper.h.inc} (100%)
rename target/hexagon/{helper.h => tcg/helper.h.inc} (100%)
rename target/hppa/{helper.h => tcg/helper.h.inc} (100%)
rename target/i386/tcg/{cc_helper_template.h => cc_helper_template.h.inc}
(100%)
rename target/i386/{helper.h => tcg/helper.h.inc} (98%)
rename target/i386/{ops_sse_header.h => tcg/ops_sse_header.h.inc} (100%)
rename target/i386/{shift_helper_template.h =>
tcg/shift_helper_template.h.inc} (100%)
rename target/loongarch/{helper.h => tcg/helper.h.inc} (100%)
rename target/m68k/{helper.h => tcg/helper.h.inc} (100%)
rename target/microblaze/{helper.h => tcg/helper.h.inc} (100%)
rename target/mips/{helper.h => tcg/helper.h.inc} (100%)
rename target/nios2/{helper.h => tcg/helper.h.inc} (100%)
rename target/openrisc/{helper.h => tcg/helper.h.inc} (100%)
rename target/ppc/{helper.h => tcg/helper.h.inc} (100%)
rename target/riscv/{helper.h => tcg/helper.h.inc} (100%)
rename target/rx/{helper.h => tcg/helper.h.inc} (100%)
rename target/s390x/{helper.h => tcg/helper.h.inc} (100%)
rename target/sh4/{helper.h => tcg/helper.h.inc} (100%)
rename target/sparc/{helper.h => tcg/helper.h.inc} (100%)
rename target/tricore/{helper.h => tcg/helper.h.inc} (100%)
rename target/xtensa/{helper.h => tcg/helper.h.inc} (100%)
rename tests/tcg/aarch64/{bti-crt.inc.c => bti-crt.c.inc} (100%)
--
2.38.1
- [PATCH v2 0/5] misc: Enforce .[ch].inc extension for re-included .c/.h files,
Philippe Mathieu-Daudé <=
- [PATCH v2 1/5] hw/remote/proxy: Remove dubious 'event_notifier-posix.c' include, Philippe Mathieu-Daudé, 2023/06/08
- [PATCH v2 2/5] tests/tcg/aarch64: Rename bti-crt.inc.c -> bti-crt.c.inc, Philippe Mathieu-Daudé, 2023/06/08
- [PATCH v2 3/5] target/arm: Rename helper template headers as '.h.inc', Philippe Mathieu-Daudé, 2023/06/08
- [PATCH v2 4/5] target/i386: Rename helper template headers as '.h.inc', Philippe Mathieu-Daudé, 2023/06/08
- [PATCH v2 5/5] target: Bulk rename 'helper.h' -> 'helper.h.inc' for all targets, Philippe Mathieu-Daudé, 2023/06/08