This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Patch to gcc/resource.c for PR target/13256, CRIS, SH, reorg,dbr, strict_low_part.


Hans-Peter Nilsson <hans-peter.nilsson@axis.com> writes:

>> From: "Zack Weinberg" <zack@codesourcery.com>
>> Date: Sun, 30 Nov 2003 22:36:18 -0800
>
>> Hans-Peter Nilsson <hans-peter.nilsson@axis.com> writes:
>> > I'm a bit distressed finding that mark_set_resources (and
>> > resource.c) is only used by the delayed-branch pass.
>> 
>> The other passes have better ways of doing the same thing (e.g.:
>> flow.c:init_propagate_block_info).  It would be nice to be rid of
>> resource.c entirely.
>
> That's one of the changes I hope to allude to by "a presumed
> reorg.c rewrite".  I didn't consider that as a fix for this bug,
> however, mostly because we're in stage 3.  Since you're a global
> maintainer replying to a patch, did you mean to imply that you'd
> prefer to approve such a change rather than this patch at this
> time?  1/2 ;-)

Nonono.  Your fix is much more appropriate for stage 3.  I cannot
approve it, though, as I do not know enough about resource.c.

zw


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]