This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, ARM][PING] Reintroduce minipool ranges for zero-extension insn patterns
- From: Ramana Radhakrishnan <ramana dot gcc at googlemail dot com>
- To: Yury Gribov <y dot gribov at samsung dot com>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, Julian Brown <julian at codesourcery dot com>, Richard Earnshaw <rearnsha at arm dot com>, Ramana Radhakrishnan <Ramana dot Radhakrishnan at arm dot com>
- Date: Tue, 28 Jan 2014 12:09:27 +0000
- Subject: Re: [PATCH, ARM][PING] Reintroduce minipool ranges for zero-extension insn patterns
- Authentication-results: sourceware.org; auth=none
- References: <52E13252 dot 2090900 at samsung dot com>
- Reply-to: ramrad01 at arm dot com
On Thu, Jan 23, 2014 at 3:16 PM, Yury Gribov <y.gribov@samsung.com> wrote:
> Hi,
>
> Julian Brown has proposed patch
> (http://gcc.gnu.org/ml/gcc-patches/2013-06/msg01191.html) for the dreadful
> push_minipool_fix error (http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49423)
> in June but it didn't seem to get enough attention.#
That patch appears to be garbled with text that appears to be from a
testcase. It needs to be rebased for trunk to deal with the additional
changes to the attributes currently.. Additionally I'm not really sure
why there is an additional load from the constant pool here - what is
the constant in this case ? Given it is atmost a 16 bit constant
surely that should be loaded with a single mov(w) instruction in armv7
land.
regards
Ramana
>
> Can we submit it?
>
> --
> Best regards,
> Yury