[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 3/6] configure: add option to enable LTO
From: |
Paolo Bonzini |
Subject: |
Re: [PATCH v2 3/6] configure: add option to enable LTO |
Date: |
Thu, 29 Oct 2020 11:19:16 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.3.1 |
On 28/10/20 19:22, Daniele Buono wrote:
> If LTO is enabled with the wrong linker/ar:
> - with the checks, it will exit at configure with an error. I can change
> this in a warning and disabling LTO if preferred.
> - without the checks compilation will fail
>
> If LTO is enabled with the wrong compiler (e.g. old gcc), you may get a
> bunch of warnings at compile time, and a binary that won't pass some of
> the tests in make check.
I think both of these count as user error or compiler bug, which we
generally don't protect against.
There is one exception. We check if the C++ compiler driver can link
object files produced by the C compiler driver; this issue arises if the
driver used for compilation (C) is GCC and the driver used for linking
(C++) is clang, because GCC and clang's sanitizer libraries are not
compatible with each other.
I think however that in this case the problem is not one of
compatibility, but just a broken install, so I think we can just ignore
and just forward b_lto.
Paolo
- Re: [PATCH v2 2/6] configure: avoid new clang 11+ warnings, (continued)
[PATCH v2 3/6] configure: add option to enable LTO, Daniele Buono, 2020/10/23
- Re: [PATCH v2 3/6] configure: add option to enable LTO, Paolo Bonzini, 2020/10/26
- Re: [PATCH v2 3/6] configure: add option to enable LTO, Daniel P . Berrangé, 2020/10/26
- Re: [PATCH v2 3/6] configure: add option to enable LTO, Daniele Buono, 2020/10/27
- Re: [PATCH v2 3/6] configure: add option to enable LTO, Daniel P . Berrangé, 2020/10/27
- Re: [PATCH v2 3/6] configure: add option to enable LTO, Daniele Buono, 2020/10/27
- Re: [PATCH v2 3/6] configure: add option to enable LTO, Paolo Bonzini, 2020/10/28
- Re: [PATCH v2 3/6] configure: add option to enable LTO, Daniele Buono, 2020/10/28
- Re: [PATCH v2 3/6] configure: add option to enable LTO,
Paolo Bonzini <=
Re: [PATCH v2 3/6] configure: add option to enable LTO, Alex Bennée, 2020/10/28
Re: [PATCH v2 3/6] configure: add option to enable LTO, Daniele Buono, 2020/10/28
[PATCH v2 4/6] cfi: Initial support for cfi-icall in QEMU, Daniele Buono, 2020/10/23
[PATCH v2 5/6] check-block: enable iotests with cfi-icall, Daniele Buono, 2020/10/23
[PATCH v2 6/6] configure: add support for Control-Flow Integrity, Daniele Buono, 2020/10/23
Re: [PATCH v2 0/6] Add support for Control-Flow Integrity, Eric Blake, 2020/10/23