[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 0/2] Replaced locks with lock guard macros
From: |
Richard Henderson |
Subject: |
Re: [PATCH v4 0/2] Replaced locks with lock guard macros |
Date: |
Sat, 28 Mar 2020 09:12:40 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.4.1 |
On 3/27/20 11:38 PM, Daniel Brodsky wrote:
> On Thu, Mar 26, 2020 at 11:01 AM Richard Henderson
> <address@hidden> wrote:
>>
>> My preference is to add -Wno-tautological-type-limit-compare in
>> configure, so we don't have to work around this same issue elsewhere
>> in the code base.
>>
>> r~
>
> What do you think would be the best way to add this? I could change
> all additions of the `-m32` flag to instead use `-m32
> -Wno-tautological-type-limit-compare` or add the flag if qemu is being
> compiled with clang and `-m32` already enabled.
I was going to add it unconditionally, with all of the other warning flags.
Except that it doesn't work -- clang-9 *still* warns. Clearly a clang bug, but
there doesn't seem to be any workaround at all except --disable-werror.
r~
- [PATCH v4 2/2] lockable: replaced locks with lock guard macros where appropriate, (continued)
- [PATCH v4 2/2] lockable: replaced locks with lock guard macros where appropriate, dnbrdsky, 2020/03/20
- Re: [PATCH v4 0/2] Replaced locks with lock guard macros, no-reply, 2020/03/20
- Re: [PATCH v4 0/2] Replaced locks with lock guard macros, Stefan Hajnoczi, 2020/03/23
- Re: [PATCH v4 0/2] Replaced locks with lock guard macros, Daniel Brodsky, 2020/03/23
- Re: [PATCH v4 0/2] Replaced locks with lock guard macros, Richard Henderson, 2020/03/23
- Re: [PATCH v4 0/2] Replaced locks with lock guard macros, Eric Blake, 2020/03/24
- Re: [PATCH v4 0/2] Replaced locks with lock guard macros, Daniel Brodsky, 2020/03/25
- Re: [PATCH v4 0/2] Replaced locks with lock guard macros, Richard Henderson, 2020/03/26
- Re: [PATCH v4 0/2] Replaced locks with lock guard macros, Daniel Brodsky, 2020/03/28
- Re: [PATCH v4 0/2] Replaced locks with lock guard macros,
Richard Henderson <=
- Re: [PATCH v4 0/2] Replaced locks with lock guard macros, Daniel Brodsky, 2020/03/30
- Re: [PATCH v4 0/2] Replaced locks with lock guard macros, Peter Maydell, 2020/03/30