[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 00/11] Ide patches
From: |
John Snow |
Subject: |
Re: [Qemu-devel] [PULL 00/11] Ide patches |
Date: |
Mon, 18 Sep 2017 13:55:16 -0400 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 |
On 09/16/2017 10:34 AM, Peter Maydell wrote:
> On 16 September 2017 at 02:03, John Snow <address@hidden> wrote:
>> The following changes since commit 5faf2d376af3cb4eb92da44c2580e08d39832caa:
>>
>> Merge remote-tracking branch 'remotes/huth/tags/check-20170915' into
>> staging (2017-09-15 20:29:44 +0100)
>>
>> are available in the git repository at:
>>
>> https://github.com/jnsnow/qemu.git tags/ide-pull-request
>>
>> for you to fetch changes up to 2a94e34d3ecef91727f467cc012587c632099d40:
>>
>> AHCI: remove DPRINTF macro (2017-09-15 20:36:18 -0400)
>>
>> ----------------------------------------------------------------
>>
>> ----------------------------------------------------------------
>
> Hi; I'm afraid this doesn't build with clang:
>
> /home/petmay01/linaro/qemu-for-merges/hw/ide/core.c:70:15: error:
> comparison of unsigned enum expression >= 0 is always true
> [-Werror,-Wtautological-compare]
> if (enval >= 0 && enval < IDE_DMA__COUNT) {
> ~~~~~ ^ ~
> 1 error generated.
>
> (It's impdef whether an enum with all positive values is
> a signed type or unsigned type, so just deleting the
> comparison against 0 would also be wrong...)
>
> thanks
> -- PMM
>
Huh, impdef in the general case, but is it undefined for gnu99? I'm
wondering why Clang can be so certain about this comparison being
useless. Is this a Clang "bug"?
--js
- [Qemu-devel] [PULL 09/11] AHCI: Rework IRQ constants, (continued)
- [Qemu-devel] [PULL 09/11] AHCI: Rework IRQ constants, John Snow, 2017/09/15
- [Qemu-devel] [PULL 05/11] IDE: add tracing for data ports, John Snow, 2017/09/15
- [Qemu-devel] [PULL 01/11] ide: ahci: unparent children buses before freeing their memory, John Snow, 2017/09/15
- [Qemu-devel] [PULL 06/11] ATAPI: Replace DEBUG_IDE_ATAPI with tracing events, John Snow, 2017/09/15
- [Qemu-devel] [PULL 07/11] IDE: replace DEBUG_AIO with trace events, John Snow, 2017/09/15
- [Qemu-devel] [PULL 11/11] AHCI: remove DPRINTF macro, John Snow, 2017/09/15
- [Qemu-devel] [PULL 08/11] AHCI: Replace DPRINTF with trace-events, John Snow, 2017/09/15
- Re: [Qemu-devel] [PULL 00/11] Ide patches, no-reply, 2017/09/15
- Re: [Qemu-devel] [PULL 00/11] Ide patches, Peter Maydell, 2017/09/16
- Re: [Qemu-devel] [PULL 00/11] Ide patches, Eric Blake, 2017/09/18
- Re: [Qemu-devel] [PULL 00/11] Ide patches,
John Snow <=
- Re: [Qemu-devel] [PULL 00/11] Ide patches, Peter Maydell, 2017/09/18
- Re: [Qemu-devel] [PULL 00/11] Ide patches, Peter Maydell, 2017/09/18
- Re: [Qemu-devel] [PULL 00/11] Ide patches, Mark Cave-Ayland, 2017/09/20
- Re: [Qemu-devel] [PULL 00/11] Ide patches, John Snow, 2017/09/20
- Re: [Qemu-devel] [PULL 00/11] Ide patches, Mark Cave-Ayland, 2017/09/20