mirror of git://gcc.gnu.org/git/gcc.git
[Ada] Secondary stack leak with access-to-subprogram
This patch modifies call resolution to recognize when the designated type of
an access-to-subprogram requires secondary stack management, and establish
the proper transient block.
------------
-- Source --
------------
-- leak7.adb
procedure Leak7 is
Max_Iterations : constant := 10_000;
function Func return String is
begin
return "Will this leak? Or will it dry?";
end Func;
type Func_Ptr is access function return String;
procedure Anonymous_Leak (Func : access function return String) is
begin
for Iteration in 1 .. Max_Iterations loop
declare
Val : constant String := Func.all;
begin null; end;
end loop;
end Anonymous_Leak;
procedure Named_Leak (Func : Func_Ptr) is
begin
for Iteration in 1 .. Max_Iterations loop
declare
Val : constant String := Func.all;
begin null; end;
end loop;
end Named_Leak;
begin
Anonymous_Leak (Func'Access);
Named_Leak (Func'Access);
end Leak7;
----------------------------
-- Compilation and output --
----------------------------
$ gnatmake -q leak7.adb
$ valgrind ./leak7 >& leak7.txt
$ grep -c "still reachable" leak7.txt
0
2018-07-31 Hristian Kirtchev <kirtchev@adacore.com>
gcc/ada/
* sem_res.adb (Resolve_Call): Establish a transient scope to
manage the secondary stack when the designated type of an
access-to-subprogram requires it.
From-SVN: r263103
This commit is contained in:
parent
51f2fc7d76
commit
51d4bdfb56
|
|
@ -1,3 +1,9 @@
|
||||||
|
2018-07-31 Hristian Kirtchev <kirtchev@adacore.com>
|
||||||
|
|
||||||
|
* sem_res.adb (Resolve_Call): Establish a transient scope to
|
||||||
|
manage the secondary stack when the designated type of an
|
||||||
|
access-to-subprogram requires it.
|
||||||
|
|
||||||
2018-07-31 Ed Schonberg <schonberg@adacore.com>
|
2018-07-31 Ed Schonberg <schonberg@adacore.com>
|
||||||
|
|
||||||
* exp_ch7.adb (Check_Unnesting_Elaboration_Code): To find local
|
* exp_ch7.adb (Check_Unnesting_Elaboration_Code): To find local
|
||||||
|
|
|
||||||
|
|
@ -6433,7 +6433,7 @@ package body Sem_Res is
|
||||||
null;
|
null;
|
||||||
|
|
||||||
elsif Expander_Active
|
elsif Expander_Active
|
||||||
and then Ekind (Nam) = E_Function
|
and then Ekind_In (Nam, E_Function, E_Subprogram_Type)
|
||||||
and then Requires_Transient_Scope (Etype (Nam))
|
and then Requires_Transient_Scope (Etype (Nam))
|
||||||
then
|
then
|
||||||
Establish_Transient_Scope (N, Manage_Sec_Stack => True);
|
Establish_Transient_Scope (N, Manage_Sec_Stack => True);
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue