Hi, Andrew

You are right, it's much simpler.

Could you give me a tip how %wd works on GCC ?

Could you send a new patch to fix the message since you have better solution ?

Thanks,

Shiva

2015-05-29 15:13 GMT+08:00 Andrew Pinski <pins...@gmail.com>:
> On Fri, May 29, 2015 at 2:33 PM, Shiva Chen <shiva0...@gmail.com> wrote:
>> Hi,
>>
>> I notice that aarch64 simd index range message not correct on 32 bit host.
>>
>> The message print by the function aarch64_simd_lane_bounds in aarch64.c.
>>
>> The function print HOST_WIDE_INT variable by %ld which is correct on
>> 64 bit host.
>>
>> However, on 32 bit host HOST_WIDE_INT would be long long.
>>
>> Therefore, print out incorrect message on 32 bit host.
>>
>> Fix the message by printing HOST_WIDE_INT variables on 32 bit host by %lld.
>
>
> Actually there is a simpler way.  Since this is error which uses the
> GCC diagnostic format you can just use %wd instead.
> So:
> error ("%Klane %ld out of range %ld - %ld", exp, lane, low, high - 1);
> Becomes:
> error ("%Klane %wd out of range %wd - %wd", exp, lane, low, high - 1);
>
> Thanks,
> Andrew Pinski
>
>>
>>
>> Shiva

Reply via email to