This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [Patch, AArch64, Obvious] Fix PR64231.
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Tejas Belagod <tejas dot belagod at arm dot com>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, Marcus Shawcroft <Marcus dot Shawcroft at arm dot com>, jakub at gcc dot gnu dot org, Andrew Pinski <pinskia at gmail dot com>
- Date: Fri, 30 Jan 2015 14:25:12 +0100
- Subject: Re: [Patch, AArch64, Obvious] Fix PR64231.
- Authentication-results: sourceware.org; auth=none
- References: <54C22A71 dot 7050603 at arm dot com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Fri, Jan 23, 2015 at 11:03:13AM +0000, Tejas Belagod wrote:
>
> Hi,
>
> This is an almost obvious patch to fix PR64231 as discovered by A. Pinksi
> and as proposed by Jakub.
>
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64231
>
> Regressions happy. OK to commit?
This is ok for trunk. We have a real bug that we need to fix, if we have
some more useful macro in the future, this can be rewritten to use that
macro together with the many other spots that would be changed for it as
well. But blocking the fix for it doesn't sound right to me.
> 2015-01-23 Tejas Belagod <tejas.belagod@arm.com>
> Andrew Pinski <pinskia@gcc.gnu.org>
> Jakub Jelinek <jakub@gcc.gnu.org>
>
> PR target/64231
> * config/aarch64/aarch64.c (aarch64_classify_symbol): Fix large
> integer typing for small model. Use IN_RANGE.
Jakub