This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: improved RTL-level if conversion using scratchpads [half-hammock edition]
- From: Bernhard Reutner-Fischer <rep dot dot dot nop at gmail dot com>
- To: Bernd Schmidt <bernds_cb1 at t-online dot de>,Sebastian Pop <sebpop at gmail dot com>,Bernd Schmidt <bschmidt at redhat dot com>
- Cc: Abe <abe_skolnik at yahoo dot com>,"gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>,Kyrill Tkachov <kyrylo dot tkachov at arm dot com>
- Date: Sat, 07 Nov 2015 13:02:39 +0100
- Subject: Re: improved RTL-level if conversion using scratchpads [half-hammock edition]
- Authentication-results: sourceware.org; auth=none
- References: <563BE9A7 dot 30803 at yahoo dot com> <563C8748 dot 3040901 at redhat dot com> <CAFk3UF_VPeUmkvz=taxCvGvoWNCLT6wWHUCiD0rOgK4+_pY-tg at mail dot gmail dot com> <563CB9F4 dot 7020609 at redhat dot com> <CAFk3UF80p+D72aLQwWw-S4=Zf1O9=0O5bX8bS1dq0H-pPj-_rg at mail dot gmail dot com> <563CD500 dot 7040304 at t-online dot de>
On November 6, 2015 5:27:44 PM GMT+01:00, Bernd Schmidt <bernds_cb1@t-online.de> wrote:
>On 11/06/2015 04:52 PM, Sebastian Pop wrote:
>
>>> opinion). If you want a half-finished redzone allocator, I can send
>you a
>>> patch.
>>
>> Yes please. Let's get it work.
>
>Here you go. This is incomplete and does not compile, but it shows the
>direction I have in mind and isn't too far off. I had a similar patch
--- a/gcc/function.c
+++ b/gcc/function.c
@@ -217,10 +217,10 @@ free_after_compilation (struct function *f)
HOST_WIDE_INT
get_frame_size (void)
{
- if (FRAME_GROWS_DOWNWARD)
- return -frame_offset;
+ if (-crtl->frame.grows_downward)
+ return -crtl->frame.frame_offset;
else
- return frame_offset;
+ return crtl->frame.frame_offset;
}
frame.grows_downward is a bool it seems and as such I wonder what the minus in the condition means or is supposed to achieve?
Something we (should?) warn about?
Just curious..
Cheers,