This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [Patch AArch64] Fix for PR62040


Changed the coding style.


2014-09-03  Guozhi Wei  <carrot@google.com>

        PR target/62040
        * gcc.target/aarch64/pr62040.c: New test.


Index: pr62040.c
===================================================================
--- pr62040.c (revision 0)
+++ pr62040.c (revision 0)
@@ -0,0 +1,21 @@
+/* { dg-do compile } */
+/* { dg-options "-g -Os" } */
+
+#include "arm_neon.h"
+
+extern void bar (int32x4_t);
+
+void
+foo ()
+{
+  int32x4x4_t rows;
+  uint64x2x2_t row01;
+
+  row01.val[0] = vreinterpretq_u64_s32 (rows.val[0]);
+  row01.val[1] = vreinterpretq_u64_s32 (rows.val[1]);
+  uint64x1_t row3l = vget_low_u64 (row01.val[0]);
+  row01.val[0] = vcombine_u64 (vget_low_u64 (row01.val[1]), row3l);
+  int32x4_t xxx = vreinterpretq_s32_u64 (row01.val[0]);
+  int32x4_t out = vtrn1q_s32 (xxx, xxx);
+  bar (out);
+}


On Wed, Sep 3, 2014 at 6:04 AM, Marcus Shawcroft
<marcus.shawcroft@gmail.com> wrote:
> On 20 August 2014 20:51, Carrot Wei <carrot@google.com> wrote:
>> Good suggestion. Add the testcase.
>>
>> thanks
>> Guozhi Wei
>>
>> 2014-08-20  Guozhi Wei  <carrot@google.com>
>>
>>         PR target/62040
>>         * gcc.target/aarch64/pr62040.c: New test.
>>
>> Index: pr62040.c
>> ===================================================================
>> --- pr62040.c (revision 0)
>> +++ pr62040.c (revision 0)
>> @@ -0,0 +1,19 @@
>> +/* { dg-do compile } */
>> +/* { dg-options "-g -Os" } */
>> +
>> +#include "arm_neon.h"
>> +
>> +extern bar(int32x4_t);
>> +
>> +void foo() {
>> +  int32x4x4_t rows;
>> +  uint64x2x2_t row01;
>> +
>> +  row01.val[0] = vreinterpretq_u64_s32(rows.val[0]);
>> +  row01.val[1] = vreinterpretq_u64_s32(rows.val[1]);
>> +  uint64x1_t row3l = vget_low_u64(row01.val[0]);
>> +  row01.val[0] = vcombine_u64(vget_low_u64(row01.val[1]), row3l);
>> +  int32x4_t xxx = vreinterpretq_s32_u64(row01.val[0]);
>> +  int32x4_t out = vtrn1q_s32 (xxx, xxx);
>> +  bar(out);
>> +}
>
>
> GNU coding style please.
>
> /Marcus


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]