[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/5] target-mips: Fix unused-function warnings
From: |
Richard Henderson |
Subject: |
Re: [Qemu-devel] [PATCH 0/5] target-mips: Fix unused-function warnings |
Date: |
Mon, 15 Sep 2014 09:12:22 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.1.0 |
On 09/14/2014 12:45 PM, Peter Maydell wrote:
> These patches fix a number of warnings in target-mips code that
> clang produces; notably, clang will complain about unused
> 'static inline' functions in .c files where gcc does not.
> Some are genuinely totally unused; a few are unused only in some
> configs and so we add ifdef guards.
>
> (I've actually had these lying around for a while but forgot to
> send them.)
>
> Peter Maydell (5):
> target-mips: Remove unused gen_load_ACX, gen_store_ACX and cpu_ACX
> target-mips/translate.c: Add ifdef guard around check_mips64()
> target-mips/dsp_helper.c: Add ifdef guards around various functions
> target-mips/dsp_helper.c: Remove unused function get_DSPControl_24()
> target-mips/op_helper.c: Remove unused do_lbu() function
>
> target-mips/dsp_helper.c | 26 ++++++++++++++++----------
> target-mips/op_helper.c | 1 -
> target-mips/translate.c | 22 +++-------------------
> 3 files changed, 19 insertions(+), 30 deletions(-)
>
Reviewed-by: Richard Henderson <address@hidden>
r~
- [Qemu-devel] [PATCH 0/5] target-mips: Fix unused-function warnings, Peter Maydell, 2014/09/14
- [Qemu-devel] [PATCH 4/5] target-mips/dsp_helper.c: Remove unused function get_DSPControl_24(), Peter Maydell, 2014/09/14
- [Qemu-devel] [PATCH 5/5] target-mips/op_helper.c: Remove unused do_lbu() function, Peter Maydell, 2014/09/14
- [Qemu-devel] [PATCH 3/5] target-mips/dsp_helper.c: Add ifdef guards around various functions, Peter Maydell, 2014/09/14
- [Qemu-devel] [PATCH 2/5] target-mips/translate.c: Add ifdef guard around check_mips64(), Peter Maydell, 2014/09/14
- [Qemu-devel] [PATCH 1/5] target-mips: Remove unused gen_load_ACX, gen_store_ACX and cpu_ACX, Peter Maydell, 2014/09/14
- Re: [Qemu-devel] [PATCH 0/5] target-mips: Fix unused-function warnings,
Richard Henderson <=