At 2013-04-02 17:50:03,"Ramana Radhakrishnan" <ramra...@arm.com> 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 <x...@marvell.com>
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
