Re: [PATCH] DECL_NO_STATIC_CHAIN -> DECL_STATIC_CHAIN

[Eric Botcazou <ebotcazou at adacore dot com>]

> I can add it, just note that this is only needed if DECL_STATIC_CHAIN flag
> is used by the Ada FE (tree-nested.c recomputes it).

It is, gnat_build_constructor calls initializer_constant_valid_p.

> Would you mind adding a testcase similar to gcc.dg/nested-fn-2.c for Ada
> that shows it matters?

Done, it segfaults at run time (before your patch) if DECL_NO_STATIC_CHAIN 
is set unconditionally in the front-end.


2009-09-24  Eric Botcazou  <ebotcazou@adacore.com>

	* gnat.dg/nested_proc.adb: New test.


-- 
Eric Botcazou

-- { dg-do run }
-- Test that a static link is correctly passed to a subprogram which is
-- indirectly called through an aggregate.

procedure Nested_Proc is

  I : Integer := 0;

  procedure P1 (X : Integer) is
  begin
    I := X;
  end;

  type Func_Ptr is access procedure (X : Integer);

  type Arr is array (1..64) of Integer;

  type Rec is record
    F : Func_Ptr;
    A : Arr;
  end record;

  procedure P2 (R : Rec) is
  begin
     R.F (1);
  end;

begin
  P2 ((F => P1'Access, A => (others => 0)));
  if I /= 1 then
    raise Program_Error;
  end if;
end;