1. Patchew
  2. QEMU
  3. View series

[PATCH v2 0/5] misc: Enforce .[ch].inc extension for re-included .c/.h files

Philippe Mathieu-Daudé posted 5 patches 11 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20230608133108.72655-1-philmd@linaro.org
Maintainers: Elena Ufimtseva <elena.ufimtseva@oracle.com>, Jagannathan Raman <jag.raman@oracle.com>, John G Johnson <john.g.johnson@oracle.com>, Richard Henderson <richard.henderson@linaro.org>, Paolo Bonzini <pbonzini@redhat.com>, Peter Maydell <peter.maydell@linaro.org>, Michael Rolnik <mrolnik@gmail.com>, "Edgar E. Iglesias" <edgar.iglesias@gmail.com>, Brian Cain <bcain@quicinc.com>, Eduardo Habkost <eduardo@habkost.net>, Song Gao <gaosong@loongson.cn>, Xiaojuan Yang <yangxiaojuan@loongson.cn>, Laurent Vivier <laurent@vivier.eu>, "Philippe Mathieu-Daudé" <philmd@linaro.org>, Aurelien Jarno <aurelien@aurel32.net>, Jiaxun Yang <jiaxun.yang@flygoat.com>, Aleksandar Rikalo <aleksandar.rikalo@syrmia.com>, Chris Wulff <crwulff@gmail.com>, Marek Vasut <marex@denx.de>, Stafford Horne <shorne@gmail.com>, Daniel Henrique Barboza <danielhb413@gmail.com>, "Cédric Le Goater" <clg@kaod.org>, David Gibson <david@gibson.dropbear.id.au>, Greg Kurz <groug@kaod.org>, Palmer Dabbelt <palmer@dabbelt.com>, Alistair Francis <alistair.francis@wdc.com>, Bin Meng <bin.meng@windriver.com>, Weiwei Li <liweiwei@iscas.ac.cn>, Liu Zhiwei <zhiwei_liu@linux.alibaba.com>, Yoshinori Sato <ysato@users.sourceforge.jp>, David Hildenbrand <david@redhat.com>, Ilya Leoshkevich <iii@linux.ibm.com>, Thomas Huth <thuth@redhat.com>, Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>, Artyom Tarasenko <atar4qemu@gmail.com>, Bastian Koppelmann <kbastian@mail.uni-paderborn.de>, Max Filippov <jcmvbkbc@gmail.com>
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%)
[PATCH v2 0/5] misc: Enforce .[ch].inc extension for re-included .c/.h files
Posted by Philippe Mathieu-Daudé 11 months ago 
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

Patchew 2.1-devel-0870f84

© 2016 - 2024 Red Hat, Inc.