qemu-arm
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] configure: Improve alias attribute check


From: Richard Henderson
Subject: Re: [PATCH] configure: Improve alias attribute check
Date: Sun, 21 Mar 2021 11:34:26 -0600
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.7.1

On 3/21/21 10:50 AM, Paolo Bonzini wrote:
    Another workaround may be to avoid compiling exec-vary.c with -flto.  I'm 
not
    sure that my meson fu is up to that.  Paolo?

You would have to define a static library.

Ok. With an extra -fno-lto flag, or can I somehow remove -flto from the library's cflags? Or unset the meson b_lto variable?

    I have filed a gcc bug report:

    https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99696
    <https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99696>

    Hopefully someone can address that before gcc 11 gets released.  At which
    point we need do nothing in qemu.  Aldy?


Good point, I can give it a shot too just to see how rusty I am... That would be the best outcome, though we would have to check LLVM as well. If const doesn't work it would indeed be prudent to include Gavin's configure check.

So, I've reproduced the testcase failure with gcc 9.3 (ubuntu 20.04) as well. Which means that there are at least two releases for which this has not worked.

I think Gavin's runtime test is unnecessary. We don't have to check the runtime results, we can just [ "$lto" = true ], and we fairly well know it'll fail.


r~



reply via email to

[Prev in Thread] Current Thread [Next in Thread]