improved RTL-level if conversion using scratchpads [half-hammock edition]
diff mbox

Message ID 76AD3AB2-2DAC-432B-8315-CCFB0BCB0883@gmail.com
State New
Headers show

Commit Message

Bernhard Reutner-Fischer Nov. 7, 2015, 12:02 p.m. UTC
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 

Something we (should?) warn about?

Just curious..
Cheers,

Patch
diff mbox

--- 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?