[PATCH][GCC][AARCH64] Use stdint integers in vect_su_add_sub.c

Matthew Malcomson matthew.malcomson@arm.com
Fri Aug 3 16:28:00 GMT 2018


On 02/08/18 20:18, James Greenhalgh wrote:
> On Tue, Jul 31, 2018 at 04:53:19AM -0500, Matthew Malcomson wrote:
>> Fixing the ilp32 issue that Christophe found.
>>
>> The existing testcase uses `long` to represent a 64 bit integer.
>> This breaks when compiled using the `-mabi=ilp32` flag.
>> We switch the use of int/long for int32_t/int64_t to avoid this problem
>> and show the requirement of a widening operation more clearly.
> Normally we try to avoid pulling more headers than we need in the testsuite.
>
> Have you seen this construct:
>
>    typedef unsigned __attribute__((mode(DI))) uint64_t;
>
> It can be a good way to ensure you have the right type for the patterns you
> are adding.
>
> Thanks,
> James
Thanks! No I hadn't seen that construct before.
Have attached a patch using that construct.

Matthew
-------------- next part --------------
A non-text attachment was scrubbed...
Name: rb9768.patch
Type: text/x-patch
Size: 1817 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20180803/5e7cb0e9/attachment.bin>


More information about the Gcc-patches mailing list