Merge lp:~rsandifo/gcc-linaro/lp-728315 into lp:gcc-linaro/4.5

Proposed by Richard Sandiford
Status: Merged
Merged at revision: 99488
Proposed branch: lp:~rsandifo/gcc-linaro/lp-728315
Merge into: lp:gcc-linaro/4.5
Diff against target: 29 lines (+10/-1)
2 files modified
ChangeLog.linaro (+5/-0)
gcc/ee.c (+5/-1)
To merge this branch: bzr merge lp:~rsandifo/gcc-linaro/lp-728315
Reviewer Review Type Date Requested Status
Andrew Stubbs (community) Approve
Review via email: mp+52103@code.launchpad.net

Description of the change

Tested on arm-linux-gnueabi/A8 with no regressions.

To post a comment you must log in.
Revision history for this message
Andrew Stubbs (ams-codesourcery) wrote :

I'll run an integration test over the weekend ....

Revision history for this message
Andrew Stubbs (ams-codesourcery) wrote :

Testing revealed no problems. This is ok.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'ChangeLog.linaro'
2--- ChangeLog.linaro 2011-03-03 13:38:27 +0000
3+++ ChangeLog.linaro 2011-03-03 17:49:03 +0000
4@@ -1,3 +1,8 @@
5+2011-03-03 Richard Sandiford <richard.sandiford@linaro.org>
6+
7+ gcc/
8+ * ee.c (reg_use_p): Handle subregs of promoted vars.
9+
10 2011-03-03 Andrew Stubbs <ams@codesourcery.com>
11
12 gcc/
13
14=== modified file 'gcc/ee.c'
15--- gcc/ee.c 2010-12-10 15:33:37 +0000
16+++ gcc/ee.c 2011-03-03 17:49:03 +0000
17@@ -209,7 +209,11 @@
18
19 *regno = REGNO (reg);
20
21- if (paradoxical_subreg_p (use))
22+ /* Non-paradoxical SUBREGs of promoted vars guarantee that the
23+ upper (elided) bits of the inner register have a particular value.
24+ For our purposes, such SUBREGs act as a full reference to the
25+ inner register. */
26+ if (paradoxical_subreg_p (use) || SUBREG_PROMOTED_VAR_P (use))
27 *size = GET_MODE_BITSIZE (GET_MODE (reg));
28 else
29 *size = subreg_lsb (use) + GET_MODE_BITSIZE (GET_MODE (use));

Subscribers

People subscribed via source and target branches