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: PR middle-end/30191 (was: [PATCH] Fix PR middle-end/28690, indexed load/store performance + reload bug)


On Wed, 2006-12-13 at 22:25 +0100, Ulrich Weigand wrote:
> The following patch attempts to work both in situations where we should
> re-recognize in order to change the insn code, and in situations where we
> cannot re-recognize because the intermediate form of the insn is not
> accepted by the target predicate.
[snip]
> Peter, could you try whether it indeed fixes your original problem?

I bootstrapped and regtested this patch on powerpc64-linux with and
without the indexed load/store patch that exposed this.  They both
passed with no errors/regressions.  However, looking into the RTL
dumps for the compiler with the indexed load/store patch, we no longer
have the REG_UNUSED note attached to the PARALLEL.  It might be due to
me updating my sources.  I'll rerun the bootstrap and regtest with an
older version of mainline I know generated the REG_UNUSED note to make
sure your change indeed handles that case.

Peter



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