On 01/28/2013 06:55 AM, Ulrich Weigand wrote:
Richard Biener wrote:
On Sun, Jan 27, 2013 at 11:26 PM, Steven Bosscher <stevenb....@gmail.com> wrote:
On Sun, Jan 27, 2013 at 11:09 PM, Georg-Johann Lay wrote:
The patch was originally worked out by Bernd Schmidt and fixed a problem
introduced in
http://gcc.gnu.org/r190252
Ironically, this revision fixes a reload problem on x86/x86_64 --
which doesn't use reload anymore now...
Does this mean the fix is rejected for 4.8?
No, just that it probably helps to add a RM to the CC list.
FWIW, it seems to me that this patch should go into 4.8, because the
bug is probably not limited to AVR.
Indeed, the fix also looks quite obvious though I know nothing about the
code at all.
Thus, ok from a RM perspective if a reload-affine person approves it.
The patch was originally by Bernd, but FWIW it looks good to me as well.
Now that I know this is a regression, I've looked at it more closely and
it looks good to me too.
George-Johann, please install this onto the trunk. Thanks,
Jeff
