[Bug target/13722] [3.4/3.5 regression] [ia64] ICE in push_secondary_reload
------- Additional Comments From wilson at specifixinc dot com 2004-01-23 22:55 -------
Subject: Re: PR 13722 candidate fix
On Thu, 2004-01-22 at 23:43, Zack Weinberg wrote:
> Under what circumstances should REG_INC notes be tagged onto the insns
> emitted here? It's currently done only for POST_MODIFYs, but this
> patch will generate POST_INC and POST_DEC as well.
I believe the answer is that we need it for all auto-inc addresses. The
REG_INC note indicates that the instruction has another side-effect
besides the sets and clobbers. Any routines that care about instruction
side-effects aren't going to notice the auto-inc address unless the
REG_INC note is there.
The fact that we only do this after reload reduces the chances of a
problem, but the rtlanal.c reg_set_p function needs it, and it is likely
that there is already some code somewhere after reload that directly or
indirectly calls reg_set_p. Thus the REG_INC notes should be added to
be safe.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=13722
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
Reply to: