[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/9] tests: cris: force inlining
From: |
Edgar E. Iglesias |
Subject: |
Re: [Qemu-devel] [PATCH 1/9] tests: cris: force inlining |
Date: |
Tue, 6 Sep 2016 23:53:46 +0200 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Mon, Sep 05, 2016 at 01:54:04PM +0200, Rabin Vincent wrote:
> From: Rabin Vincent <address@hidden>
>
> The CRIS tests expect that functions marked inline are always inline.
> With newer versions of GCC, building them results warnings like the
> following and spurious failures when they are run.
>
> In file included from tests/tcg/cris/check_moveq.c:5:0:
> tests/tcg/cris/crisutils.h:66:20: warning: inlining failed in call to
> 'cris_tst_cc.constprop.0': call is unlikely and code size would grow
> [-Winline]
> tests/tcg/cris/check_moveq.c:28:13: warning: called from here [-Winline]
>
> Use the always_inline attribute when building them to fix this.
Hi Rabin,
Do you happen to have a public git tree/branch with these changes?
Best regards,
Edgar
>
> Signed-off-by: Rabin Vincent <address@hidden>
> ---
> tests/tcg/cris/sys.h | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/tests/tcg/cris/sys.h b/tests/tcg/cris/sys.h
> index c5f88e1..b1bf4c5 100644
> --- a/tests/tcg/cris/sys.h
> +++ b/tests/tcg/cris/sys.h
> @@ -3,6 +3,8 @@
> #define STRINGIFY(x) #x
> #define TOSTRING(x) STRINGIFY(x)
>
> +#define inline inline __attribute__((always_inline))
> +
> #define CURRENT_LOCATION __FILE__ ":" TOSTRING(__LINE__)
>
> #define err() \
> --
> 2.1.4
>
- [Qemu-devel] [PATCHv2 2/8] tests: cris: fix syscall inline asm, (continued)
[Qemu-devel] [PATCH 5/9] target-cris: sync CC state at load/stores., Rabin Vincent, 2016/09/05
[Qemu-devel] [PATCH 6/9] target-cris: reduce v32isms from v10 log dumps, Rabin Vincent, 2016/09/05
[Qemu-devel] [PATCH 4/9] tests: cris: remove check_time1, Rabin Vincent, 2016/09/05
Re: [Qemu-devel] [PATCH 1/9] tests: cris: force inlining,
Edgar E. Iglesias <=
Re: [Qemu-devel] [PATCH 1/9] tests: cris: force inlining, Peter Maydell, 2016/09/08