Re: [PATCH, ARM, iWMMXT] PR target/54338 - Include IWMMXT_GR_REGS in ALL_REGS
> If so is the backport OK for 4.8? I am happy for it to go to 4.8 provided it's tested on 4.8 and the release managers don't object. Ramana > > Thanks, > > Matt > > > -- > Matthew Gretton-Dann > Toolchain Working Group, Linaro
RE: [PATCH, ARM, iWMMXT] PR target/54338 - Include IWMMXT_GR_REGS in ALL_REGS
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, > >>> 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_CONTENTS. > >>> It seems there is no reason to exclude the IWMMXT_GR_REGS from > ALL_REGS as IWMMXT_GR_REGS are the real registers. > >>> This patch simply 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. > >> > >> Testing just with arm.exp is not enough. > >> > >> Ok if no regressions running the entire regression testsuite for C > >> and > >> C++ for arm*-*-*eabi with an iwmmxt configuration. > > > > Hi Ramana, > > > >I run the full dejagnu test with -march=iwmmxt2 specified in the whole > progress for this patch. > >No regression but a lot of new pass found in the test. > >Please help to commit it. > > > > ChangeLog > > > > 2013-04-02 Xinyu Qi > > > > PR target/54338 > > * config/arm/arm.h (REG_CLASS_CONTENTS): Include > IWMMXT_GR_REGS in ALL_REGS. > > It looks to me as if this should also be applied to the 4.8 branch - Xinyu do > you > agree? Hi Matt, I'm OK with the patch applied to the 4.8 branch. (But I only tested the patch for the trunk.) Thanks, Xinyu > > If so is the backport OK for 4.8? > > Thanks, > > Matt > > > -- > Matthew Gretton-Dann > Toolchain Working Group, Linaro
Re: [PATCH, ARM, iWMMXT] PR target/54338 - Include IWMMXT_GR_REGS in ALL_REGS
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, 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_CONTENTS. It seems there is no reason to exclude the IWMMXT_GR_REGS from ALL_REGS as IWMMXT_GR_REGS are the real registers. This patch simply 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. Testing just with arm.exp is not enough. Ok if no regressions running the entire regression testsuite for C and C++ for arm*-*-*eabi with an iwmmxt configuration. Hi Ramana, I run the full dejagnu test with -march=iwmmxt2 specified in the whole progress for this patch. No regression but a lot of new pass found in the test. Please help to commit it. ChangeLog 2013-04-02 Xinyu Qi PR target/54338 * config/arm/arm.h (REG_CLASS_CONTENTS): Include IWMMXT_GR_REGS in ALL_REGS. It looks to me as if this should also be applied to the 4.8 branch - Xinyu do you agree? If so is the backport OK for 4.8? Thanks, Matt -- Matthew Gretton-Dann Toolchain Working Group, Linaro
Re: [PATCH, ARM, iWMMXT] PR target/54338 - Include IWMMXT_GR_REGS in ALL_REGS
ChangeLog 2013-04-02 Xinyu Qi PR target/54338 * config/arm/arm.h (REG_CLASS_CONTENTS): Include IWMMXT_GR_REGS in ALL_REGS. Thanks, Xinyu Thanks now applied to trunk. For the future please consider creating patches at the top level directory. Makes it easier for application by someone else :) . regards Ramana
Re: [PATCH, ARM, iWMMXT] PR target/54338 - Include IWMMXT_GR_REGS in ALL_REGS
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_CONTENTS. >>It seems there is no reason to exclude the IWMMXT_GR_REGS from ALL_REGS >> as IWMMXT_GR_REGS are the real registers. >>This patch simply 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. > >Testing just with arm.exp is not enough. > >Ok if no regressions running the entire regression testsuite for C and >C++ for arm*-*-*eabi with an iwmmxt configuration. Hi Ramana, I run the full dejagnu test with -march=iwmmxt2 specified in the whole progress for this patch. No regression but a lot of new pass found in the test. Please help to commit it. ChangeLog 2013-04-02 Xinyu Qi PR target/54338 * config/arm/arm.h (REG_CLASS_CONTENTS): Include IWMMXT_GR_REGS in ALL_REGS. Thanks, Xinyu IWMMXT_GR_REGS.diff Description: IWMMXT_GR_REGS.diff
Re: [PATCH, ARM, iWMMXT] PR target/54338 - Include IWMMXT_GR_REGS in ALL_REGS
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_CONTENTS. It seems there is no reason to exclude the IWMMXT_GR_REGS from ALL_REGS as IWMMXT_GR_REGS are the real registers. This patch simply 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. Testing just with arm.exp is not enough. Ok if no regressions running the entire regression testsuite for C and C++ for arm*-*-*eabi with an iwmmxt configuration. Thanks Ramana
[PATCH, ARM, iWMMXT] PR target/54338 - Include IWMMXT_GR_REGS in ALL_REGS
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_CONTENTS. It seems there is no reason to exclude the IWMMXT_GR_REGS from ALL_REGS as IWMMXT_GR_REGS are the real registers. This patch simply 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/arm/arm.h (REG_CLASS_CONTENTS): Include IWMMXT_GR_REGS in ALL_REGS. OK? Thanks, Xinyu IWMMXT_GR_REGS.diff Description: IWMMXT_GR_REGS.diff