On Tue, Nov 10, 2015 at 12:06 PM, Markus Armbruster wrote:
> Steven Noonan writes:
>
>> On Tue, Nov 10, 2015 at 10:52 AM, Markus Armbruster
>> wrote:
>>> Peter Maydell writes:
>>>
On 10 November 2015 at 17:33, Paolo Bonzini wrote:
>
>
> On 10/11/2015 16:57, Stefan Hajnoczi wr
On 10/11/2015 20:51, Steven Noonan wrote:
> I don't read this warning as "clang will do crazy things with your
> code eventually". Clang has always been very verbose when it comes to
> undefined behavior, and I don't think that's really a bad thing to do.
Sure, but it doesn't belong in -Wall. I
Steven Noonan writes:
> On Tue, Nov 10, 2015 at 10:52 AM, Markus Armbruster wrote:
>> Peter Maydell writes:
>>
>>> On 10 November 2015 at 17:33, Paolo Bonzini wrote:
On 10/11/2015 16:57, Stefan Hajnoczi wrote:
> clang 3.7.0 on x86_64 warns about the following:
>
>
On Tue, Nov 10, 2015 at 10:52 AM, Markus Armbruster wrote:
> Peter Maydell writes:
>
>> On 10 November 2015 at 17:33, Paolo Bonzini wrote:
>>>
>>>
>>> On 10/11/2015 16:57, Stefan Hajnoczi wrote:
clang 3.7.0 on x86_64 warns about the following:
disas/arm.c:1782:17: warning: shift
Peter Maydell writes:
> On 10 November 2015 at 17:33, Paolo Bonzini wrote:
>>
>>
>> On 10/11/2015 16:57, Stefan Hajnoczi wrote:
>>> clang 3.7.0 on x86_64 warns about the following:
>>>
>>> disas/arm.c:1782:17: warning: shifting a negative signed value is
>>> undefined [-Wshift-negative-value]
On 10/11/2015 18:48, Peter Maydell wrote:
> It could be they're attempting to warn us now about the possibility
> that in a future version of clang they will start using this UB
> to optimize with.
Good luck to them.
Paolo
On 10 November 2015 at 17:33, Paolo Bonzini wrote:
>
>
> On 10/11/2015 16:57, Stefan Hajnoczi wrote:
>> clang 3.7.0 on x86_64 warns about the following:
>>
>> disas/arm.c:1782:17: warning: shifting a negative signed value is
>> undefined [-Wshift-negative-value]
>> imm |= (-1 << 7);
>>
On 10/11/2015 16:57, Stefan Hajnoczi wrote:
> clang 3.7.0 on x86_64 warns about the following:
>
> disas/arm.c:1782:17: warning: shifting a negative signed value is undefined
> [-Wshift-negative-value]
> imm |= (-1 << 7);
> ~~ ^
>
> Note that this patch preserves the tab inde
clang 3.7.0 on x86_64 warns about the following:
disas/arm.c:1782:17: warning: shifting a negative signed value is undefined
[-Wshift-negative-value]
imm |= (-1 << 7);
~~ ^
Note that this patch preserves the tab indent in this source file
because the surrounding code still uses
