This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, ARM] Fix length attributes for sync.md patterns
- From: "Ulrich Weigand" <uweigand at de dot ibm dot com>
- To: rearnsha at arm dot com (Richard Earnshaw)
- Cc: gcc-patches at gcc dot gnu dot org (gcc-patches at gcc dot gnu dot org), patches at linaro dot org (patches at linaro dot org), ramana dot radhakrishnan at linaro dot org (ramana dot radhakrishnan at linaro dot org)
- Date: Tue, 17 Jul 2012 17:17:07 +0200 (CEST)
- Subject: Re: [PATCH, ARM] Fix length attributes for sync.md patterns
Richard Earnshaw wrote:
> Hmm, I wonder if we should just unconditionally call split_all_insns()
> at the start of md_reorg when -O0. This would address your problem, but
> have the added benefit that the length calculations would be more
> accurate. We're going to have to split the insns anyway during output,
> so why not get it over and done with...
Yes, that's already done on various other targets (including s390).
[ Note that you need split_all_insns_noflow at this stage. ]
I've noticed many other forced-split insn patterns that also carry
a length attribute -- if we do the change you suggest, I guess we
should then also remove those attributes?
Bye,
Ulrich
--
Dr. Ulrich Weigand
GNU Toolchain for Linux on System z and Cell BE
Ulrich.Weigand@de.ibm.com