|
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~
[Prev in Thread] | Current Thread | [Next in Thread] |