[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 1/5] tcg: Split CONFIG_ATOMIC128

From: Richard Henderson
Subject: Re: [Qemu-devel] [PATCH 1/5] tcg: Split CONFIG_ATOMIC128
Date: Mon, 20 Aug 2018 12:26:48 -0700
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1

On 08/17/2018 09:42 AM, Emilio G. Cota wrote:
>> +/* Through gcc 8, aarch64 has no support for 128-bit at all.  */
>> +static inline Int128 atomic16_cmpxchg(Int128 *ptr, Int128 cmp, Int128 new)
>> +{
>> +    uint64_t cmpl = cmp, cmph = cmp >> 64;
>> +    uint64_t newl = new, newh = new >> 64;
> Here I'd use int128_getlo/hi, since we're not checking for
> CONFIG_INT128 (I'm thinking of old compilers here)

I suppose, but this is aarch64 -- there are no really old compilers.
The oldest is probably gcc 4.8, which already has __int128_t.

Thanks for the other catches too.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]