At 2013-05-24 15:19:36,"Chung-Ju Wu" wrote:
> 2013/5/24 Xinyu Qi :
> > Hi,
> >
> > For this simple case, compiled with option -march=iwmmxt -O, #define
> > N 64 signed int b[N]; signed long long j[N], d[N]; void foo (void) {
> > int i;
> > f
l dejagnu test. No regression.
Is this fix proper?
OK for trunk?
ChangeLog
gcc/
2013-05-24 Xinyu Qi
* config/arm/arm.h (SECONDARY_OUTPUT_RELOAD_CLASS): Check
IWMMXT_GR_REGS.
(SECONDARY_INPUT_RELOAD_CLASS): Likewise.
testsuite/
2013-05-24 Xinyu Qi
* gcc.target/ar
At 2013-04-30 18:45:42,"Matthew Gretton-Dann"
wrote:
> Hi,
>
> On 08/04/13 06:28, Xinyu Qi wrote:
> > At 2013-04-02 17:50:03,"Ramana Radhakrishnan"
> wrote:
> >> On 04/02/13 10:40, Xinyu Qi wrote:
> >>> Hi,
> >>>
At 2013-04-02 17:50:03,"Ramana Radhakrishnan" wrote:
>On 04/02/13 10:40, Xinyu Qi wrote:
>> Hi,
>>According to Vladimir Makarov's analysis, the root cause of PR
>> target/54338 is that ALL_REGS doesn't contain IWMMXT_GR_REGS in
>> REG_CLASS
mply makes ALL_REGS include IWMMXT_GR_REGS to fix this PR.
Since the test case gcc.target/arm/mmx-2.c would fail for the same reason and
become pass with this fix, no extra test case need to be add.
Pass arm.exp test. Patch attached.
ChangeLog
2013-04-02 Xinyu Qi
* config
>At 2013-01-22 19:58:43,"Ramana Radhakrishnan" wrote:>
> > On 01/22/13 09:21, Xinyu Qi wrote:
> > > Ping,
> > >
> > > Fix ChangeLog
> >
> > The ChangeLog format includes .
> >
> >
> >
> > If you want a
At 2013-01-22 19:58:43,"Ramana Radhakrishnan" wrote:>
> On 01/22/13 09:21, Xinyu Qi wrote:
> > Ping,
> >
> > Fix ChangeLog
>
> The ChangeLog format includes .
>
>
>
> If you want a patch accepted in the future, please help by creat
At 2013-01-22 19:58:43,"Ramana Radhakrishnan" wrote:
> On 01/22/13 09:21, Xinyu Qi wrote:
> > Ping,
> >
> > Fix ChangeLog
>
> The ChangeLog format includes .
>
>
>
> If you want a patch accepted in the future, please help by creating the
&
Ping,
Fix ChangeLog
* config/arm/arm.h (FIRST_IWMMXT_GR_REGNUM): Add comment.
* config/arm/iwmmxt.md (WCGR0): Update.
(WCGR1, WCGR2, WCGR3): Likewise.
> Hi,
>
> It is necessary to sync the constants WCGR0 to WCGR3 in iwmmxt.md with
> the IWMMXT_GR_REGNUM in arm.h.
>
>
Hi,
It is necessary to sync the constants WCGR0 to WCGR3 in iwmmxt.md
with the IWMMXT_GR_REGNUM in arm.h.
ChangeLog
* config/arm/arm.h (FIRST_IWMMXT_GR_REGNUM): Add comment.
* config/arm/iwmmxt.md (WCGR0, WCGR1): Update.
* config/arm/iwmmxt.md (WCGR2, WCGR3): Likewise.
> -Original Message-
> From: Matt Turner [mailto:matts...@gmail.com]
> To: Xinyu Qi
> Cc: Ramana Radhakrishnan; GCC Patches
> Subject: Re: [PING] iwMMXt patches
>
> On Thu, May 3, 2012 at 12:59 AM, Xinyu Qi wrote:
> >> From: Matt Turner [mailto:matts..
> From: Matt Turner [mailto:matts...@gmail.com]
> To: Xinyu Qi
> Cc: Ramana Radhakrishnan; GCC Patches
> Subject: Re: [PING] iwMMXt patches
>
> On Tue, Apr 17, 2012 at 4:17 PM, Matt Turner wrote:
> > Are these patches ready to go in? It looks like they were ack'd.
PING
At 2012-02-03 10:09:55,"Xinyu Qi" wrote:
> PING
>
> http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01786.html
>
> At 2011-12-29 14:12:44,"Xinyu Qi" wrote:
> > At 2011-12-22 17:53:45,"Richard Earnshaw" wrote:
> > > On 22/12/11 06
PING
At 2012-02-03 10:07:56,"Xinyu Qi" wrote:
> PING
>
> http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01789.html
>
> At 2011-12-29 14:25:23,"Xinyu Qi" wrote:
> > > At 2011-11-24 09:27:04,"Xinyu Qi" wrote:
> > > > At 2011-11-
PING
At 2012-02-03 10:05:22,"Xinyu Qi" wrote:
> PING
>
> http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01788.html
>
> At 2011-12-29 14:22:50,"Xinyu Qi" wrote:
> > * config/arm/mmintrin.h: Use __IWMMXT__ to enable iWMMXt
> > intrinsics.
>
PING
At 2012-02-03 10:02:28,"Xinyu Qi" wrote:
> PING
>
> http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01787.html
>
> At 2011-12-29 14:20:20,"Xinyu Qi" wrote:
> > > At 2011-12-15 00:47:48,"Richard Earnshaw" wrote:
>
PING
http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01786.html
At 2011-12-29 14:12:44,"Xinyu Qi" wrote:
> At 2011-12-22 17:53:45,"Richard Earnshaw" wrote:
> > On 22/12/11 06:38, Xinyu Qi wrote:
> > > At 2011-12-15 01:32:13,"Richard Earnshaw"
PING
http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01789.html
At 2011-12-29 14:25:23,"Xinyu Qi" wrote:
> > At 2011-11-24 09:27:04,"Xinyu Qi" wrote:
> > > At 2011-11-19 07:08:22,"Ramana Radhakrishnan"
> > > wrote:
> > &
PING
http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01788.html
At 2011-12-29 14:22:50,"Xinyu Qi" wrote:
> * config/arm/mmintrin.h: Use __IWMMXT__ to enable iWMMXt
> intrinsics.
> Use __IWMMXT2__ to enable iWMMXt2 intrinsics.
> Use C name-mangling for intri
PING
http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01787.html
At 2011-12-29 14:20:20,"Xinyu Qi" wrote:
> > At 2011-12-15 00:47:48,"Richard Earnshaw" wrote:
> > > On 14/07/11 08:35, Xinyu Qi wrote:
> > > >>> Hi,
> >
> At 2011-11-24 09:27:04,"Xinyu Qi" wrote:
> > At 2011-11-19 07:08:22,"Ramana Radhakrishnan"
> > wrote:
> > > On 20 October 2011 08:39, Xinyu Qi wrote:
> > > > Ping
> > > >
> > > > http://gcc.gnu.org/ml/g
* config/arm/mmintrin.h: Use __IWMMXT__ to enable iWMMXt intrinsics.
Use __IWMMXT2__ to enable iWMMXt2 intrinsics.
Use C name-mangling for intrinsics.
(__v8qi): Redefine.
(_mm_cvtsi32_si64, _mm_andnot_si64, _mm_sad_pu8): Revise.
(_mm_sad_pu16, _mm_ali
> At 2011-12-15 00:47:48,"Richard Earnshaw" wrote:
> > On 14/07/11 08:35, Xinyu Qi wrote:
> > >>> Hi,
> > >>>
> > >>> It is the first part of iWMMXt maintenance.
> > >>>
> > >>> *config/arm/a
At 2011-12-22 17:53:45,"Richard Earnshaw" wrote:
> On 22/12/11 06:38, Xinyu Qi wrote:
> > At 2011-12-15 01:32:13,"Richard Earnshaw" wrote:
> >> On 24/11/11 01:33, Xinyu Qi wrote:
> >>> Hi Ramana,
> >>>
> >>> I solve th
At 2011-10-20 20:36:53,"Ramana Radhakrishnan"
wrote:
> On 20 October 2011 08:42, Xinyu Qi wrote:
> > Ping
> >
> > http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01106.html
> > Inde
At 2011-11-24 09:27:04,"Xinyu Qi" wrote:
> At 2011-11-19 07:08:22,"Ramana Radhakrishnan"
> wrote:
> > On 20 October 2011 08:39, Xinyu Qi wrote:
> > > Ping
> > >
> > > http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01103.html
> &
At 2011-12-15 01:32:13,"Richard Earnshaw" wrote:
> On 24/11/11 01:33, Xinyu Qi wrote:
> > Hi Ramana,
> >
> > I solve the conflict, please try again. The new diff is attached.
> >
> > Thanks,
> > Xinyu
> >
> > At 2011-11-19
At 2011-12-15 00:47:48,"Richard Earnshaw" wrote:
> On 14/07/11 08:35, Xinyu Qi wrote:
> >>> Hi,
> >>>
> >>> It is the first part of iWMMXt maintenance.
> >>>
> >>> *config/arm/arm.c (arm_option_override):
> >>>
I've not yet investigated why.
>
> Can you have a look ?
>
> cheers
> Ramana
>
> On 26 September 2011 04:22, Xinyu Qi wrote:
> > Ping.
> >
> > http://gcc.gnu.org/ml/gcc-patches/2011-09/msg00279.html
> >
> > * config/arm/arm.c (a
At 2011-11-19 07:08:22,"Ramana Radhakrishnan"
wrote:
> On 20 October 2011 08:39, Xinyu Qi wrote:
> > Ping
> >
> > http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01103.html
> >
> > * config/arm/arm.c (enum arm_builtins): Revise built-in fcode.
At 2011-11-22 18:37:16,"Ramana Radhakrishnan"
wrote:
> >>
> >> Though I don't fully understand this patch, I think it is unnecessary to
> consider turn on Thumb2 for iwmmxt,
> >> because there is no chip that supports WMMX instructions with thumb2
> mode.
>
> Xinyu: I seem to have mis-remembere
At 2011-11-19 07:11:17,"Ramana Radhakrishnan"
wrote:
> On 10 November 2011 18:07, Sameera Deshpande
> wrote:
> > Please find attached the reworked patch.
>
> OK but for a very small bit .
>
> I'll note that we need to add support for the iwMMXt registers but the
> attached patch (untested) sho
Ping
http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01106.html
* config/arm/t-arm (MD_INCLUDES): Add marvell-f-iwmmxt.md.
* config/arm/marvell-f-iwmmxt.md: New file.
* config/arm/arm.md (marvell-f-iwmmxt.md): Include.
Ping
http://gcc.gnu.org/ml/gcc-patches/2011-09/msg00279.html
* config/arm/arm.c (arm_output_iwmmxt_shift_immediate): New function.
(arm_output_iwmmxt_tinsr): Likewise.
* config/arm/arm-protos.h (arm_output_iwmmxt_shift_immediate): Declare.
(arm_output_iwmmxt_tinsr)
Ping
http://gcc.gnu.org/ml/gcc-patches/2011-08/msg01963.html
* config/arm/mmintrin.h: Revise.
Ping
http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01103.html
* config/arm/arm.c (enum arm_builtins): Revise built-in fcode.
(builtin_description bdesc_2arg): Revise built in declaration.
(builtin_description bdesc_1arg): Likewise.
(arm_init_iwmmxt_builtins): Revise
Ping
http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01100.html
* config/arm/arm.c (arm_option_override): Enable use of iWMMXt with VFP.
Disable use of iwMMXt and Neon.
(arm_expand_binop_builtin): Accept VOIDmode op.
* config/arm/arm.md (*arm_movdi): Remove check for
Ping.
http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01106.html
* config/arm/t-arm (MD_INCLUDES): Add marvell-f-iwmmxt.md.
* config/arm/marvell-f-iwmmxt.md: New file.
* config/arm/arm.md (marvell-f-iwmmxt.md): Include.
At 2011-07-29 11:09:37,"Xinyu Qi" wro
.
(UNSPEC_WMADDS, UNSPEC_WMADDU): Delete.
(UNSPEC_WALIGNI): New unspec.
* config/arm/t-arm (MD_INCLUDES): Add iwmmxt2.md.
At 2011-09-05 17:55:34,"Xinyu Qi" wrote:
> At 2011-08-18 10:21:01,"Ramana Radhakrishnan"
> wrote:
> > On 14 July 2011 0
): Revise built in initialization.
(arm_expand_builtin): Revise built in expansion.
At 2011-07-29 11:03:17,"Xinyu Qi" wrote:
> Ping.
>
> http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01103.html
>
> At 2011-07-14 15:38:47,"Xinyu Qi" wrote:
> > > Hi
Ping
http://gcc.gnu.org/ml/gcc-patches/2011-08/msg01963.html
* config/arm/mmintrin.h: Revise.
At 2011-08-24 16:14:30,"Xinyu Qi" wrote:
> At 2011-08-18 09:33:27,"Ramana Radhakrishnan"
> wrote:
> > On 6 July 2011 11:11, Xinyu Qi wrote:
> > >
TARGET_IWMMXT.
(*arm_movsi_insn): Likewise.
(iwmmxt.md): Include earlier.
At 2011-07-29 10:57:39,"Xinyu Qi" wrote: >
> Ping.
>
> http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01100.html
>
At 2011-08-18 10:21:01,"Ramana Radhakrishnan"
wrote:
> On 14 July 2011 08:45, Xinyu Qi wrote:
> >> Hi,
> >>
> >> It is the fourth part of iWMMXt maintenance.
> >>
>
> Can this be broken down further. ? I'll have to do this again but
iWMMXt2 intrinsics and the bugs could be found
everywhere since it is lack of maintenance for a long time.
> > I have seen much more extensive patches from Xinyu Qi, but I do not
> > suppose that they will be available in gcc 4.6.
The patches I submitted have some conflict with 4.6 code base.
Thanks,
Xinyu
> >
> > Thanks,
> > Matt Turner
>
>
> -Original Message-
> From: Ramana Radhakrishnan [mailto:ramana.radhakrish...@linaro.org]
At 2011-08-18 09:29:34,"Ramana Radhakrishnan"
wrote:
> Hi ,
>
> Sorry about the delayed review - It's taken me a while to get back to this .
>
> On
Ping.
http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01106.html
At 2011-07-14 15:50:55,"Xinyu Qi" wrote:
>
> > Hi,
> >
> > It is the fifth part of iWMMXt maintenance.
> >
>
> *config/arm/t-arm (MD_INCLUDES): Add marvell-f-iwmmxt.md.
> *config/ar
Ping.
http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01105.html
At 2011-07-14 15:45:09,"Xinyu Qi" wrote:
> > Hi,
> >
> > It is the fourth part of iWMMXt maintenance.
> >
>
> Since "*cond_iwmmxt_movsi_insn" would be got rid of
Ping.
http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01103.html
At 2011-07-14 15:38:47,"Xinyu Qi" wrote:
> > Hi,
> >
> > It is the third part of iWMMXt maintenance.
> >
>
> *config/arm/arm.c (enum arm_builtins): Built-in fcode.
> (builti
Ping.
http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01101.html
At 2011-07-14 15:38:04,"Xinyu Qi" wrote:
> > Hi,
> >
> > It is the second part of iWMMXt maintenance.
>
>
> *config/arm/mmintrin.h: Revise.
>
> Thanks,
> Xinyu
Ping.
http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01100.html
At 2011-07-14 15:35:52,"Xinyu Qi" wrote:
> > > Hi,
> > >
> > > It is the first part of iWMMXt maintenance.
> > >
> > > *config/arm/arm.c (arm_option_override):
> >
> Hi,
>
> It is the fifth part of iWMMXt maintenance.
>
*config/arm/t-arm (MD_INCLUDES): Add marvell-f-iwmmxt.md.
*config/arm/marvell-f-iwmmxt.md: New file.
*config/arm/arm.md (marvell-f-iwmmxt.md): Include.
Thanks,
Xinyu
5_pipeline.diff
Description: 5_pipeline.diff
> Hi,
>
> It is the fourth part of iWMMXt maintenance.
>
Since "*cond_iwmmxt_movsi_insn" would be got rid of soon, I keep it unchanged.
*config/arm/arm.c (arm_output_iwmmxt_shift_immediate): New function.
(arm_output_iwmmxt_tinsr): Ditto.
*config/arm/arm-protos.h (arm_output_iwmmxt_shift_immed
> Hi,
>
> It is the third part of iWMMXt maintenance.
>
*config/arm/arm.c (enum arm_builtins): Built-in fcode.
(builtin_description bdesc_2arg): Built in declare.
(builtin_description bdesc_1arg): Ditto.
(arm_init_iwmmxt_builtins): Built in initialize.
(arm_expand_builtin): Built in expand.
>
> Hi,
>
> It is the second part of iWMMXt maintenance.
*config/arm/mmintrin.h: Revise.
Thanks,
Xinyu
2_mmintrin.diff
Description: 2_mmintrin.diff
> > Hi,
> >
> > It is the first part of iWMMXt maintenance.
> >
> > *config/arm/arm.c (arm_option_override):
> > Enable iWMMXt with VFP. iWMMXt and NEON are incompatible.
> iWMMXt unsupported under Thumb-2 mode.
> > (arm_expand_binop_builtin): Accept immediate op (with mode VOID)
> > *config/a
Hi,
It is the fifth part of iWMMXt maintenance.
*config/arm/marvell-f-iwmmxt.md: New file. Add Marvell WMMX pipeline
description.
Thanks,
Xinyu
5_pipeline.diff
Description: 5_pipeline.diff
Hi,
It is the fourth part of iWMMXt maintenance.
*config/arm/arm.c (arm_output_iwmmxt_shift_immediate): New function. Serving
for WMMX immediate shift insn output.
(arm_output_iwmmxt_tinsr): New function. Serving for tinsr insn output.
*config/arm/arm-protos.h: Add new functions protos.
*config
Hi,
It is the third part of iWMMXt maintenance.
*config/arm/arm.c (enum arm_builtins): Add/fix iWMMXt built-in fcode.
(builtin_description bdesc_2arg): Add/fix iWMMXt built in.
(builtin_description bdesc_1arg): Same.
(arm_init_iwmmxt_builtins): Define new built-in function type. Add/fix iWMMX
Hi,
It is the first part of iWMMXt maintenance.
*config/arm/arm.c (arm_option_override):
Enable iWMMXt with VFP. iWMMXt and NEON are incompatible. iWMMXt unsupported
under Thumb-2 mode.
(arm_expand_binop_builtin): Accept immediate op (with mode VOID)
*config/arm/arm.md:
Resettle include locat
Hi,
Since the patch of iWMMXt intrinsics maintenance and pipeline description is
too big to review, I subdivide it into five parts.
1 ARM generic code change
2 iWMMXt intrinsic head file change.
3 iWMMXt built in define and expand
4 WMMX machine description
5 WMMX pipeline description
These fiv
Hi,
It is the second part of iWMMXt maintenance.
*config/arm/mmintrin.h:
Revise the iWMMXt intrinsics head file. Fix some intrinsics and add some new
intrinsics.
Thanks,
Xinyu
2_mmintrin.diff
Description: 2_mmintrin.diff
Hi,
This patch adds iWMMXt intrinsics documentation into gcc doc.
doc/extend.texi: Update iwmmxt intrinsics doc.
doc/arm-iwmmxt-intrinsics.texi: New.
Thanks,
Xinyu
iwmmxt_doc.patch
Description: iwmmxt_doc.patch
Hi,
This patch adds some simple iWMMXt intrinsic test cases.
gcc.target/arm/iwmmxt/i_shift_error.c: New.
gcc.target/arm/iwmmxt/iwmmxt.exp: New.
gcc.target/arm/iwmmxt/i_wshufh_error.c: New.
gcc.target/arm/iwmmxt/i_merge_si64_error.c: New.
gcc.target/arm/iwmmxt/i_waligni_error.c: New.
gcc.target/ar
Hi,
This patch maintains iWMMXt intrinsics code, and adds WMMX pipeline description.
*config/arm/arm.c (arm_option_override): Enable iWMMt with VFP
(enum arm_builtins): Add/fix iWMMXT/iWMMXT2 intrinsics.
(builtin_description bdesc_2arg): Same.
(builtin_description bdesc_1arg): Same.
(arm_init
Hi, Ramana and Joseph,
Thank you for your reviewing! Sorry for the late response.
Before I submit the new modified patch, I want to make something more specific.
> The -mwmmxt option is not acceptable as it stands today. IIRC the msimd
> option was the plan long term when we talked about this
Hi,
This patch maintains iWMMXT intrinsics code,
adds WMMX pipeline description
and supports iWMMXT auto-vectorization.
Ran Arm testsuite on arm-linux-gnueabi.
*gcc/config/arm/elf.h: Add option -mwmmxt.
*gcc/config/arm/arm.opt: Same.
*gcc/config/arm/arm.c (arm_option_override): Same.
(arm_coproc
66 matches
Mail list logo
