[PATCH, ARM, iWMMXT] PR target/54338 - Include IWMMXT_GR_REGS in ALL_REGS

[Xinyu Qi]

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

	PR target/54338
	* config/arm/arm.h (REG_CLASS_CONTENTS): Include IWMMXT_GR_REGS in ALL_REGS.

Thanks,
Xinyu

-------------- next part --------------
A non-text attachment was scrubbed...
Name: IWMMXT_GR_REGS.diff
Type: application/octet-stream
Size: 1359 bytes
Desc: IWMMXT_GR_REGS.diff
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20130408/d0db2580/attachment.obj>