diff mbox series

[committed] d: Disable the D runtime garbage collector after initializing (PR103520)

Message ID 20211202004759.426907-1-ibuclaw@gdcproject.org
State Committed
Commit fa10faa72d8e9e97b6e564f8b0a7b602538d0da0
Headers
Series [committed] d: Disable the D runtime garbage collector after initializing (PR103520) |

Commit Message

Iain Buclaw Dec. 2, 2021, 12:47 a.m. UTC 
  Hi,

This patch disables the D runtime garbage collector after initializing.

Not all targets that support building libdruntime have a stable garbage
collector, so to avoid running into problems where live memory allocated
by the D GC is freed, disable all in-flight collections until a time
when scanning is more reliably implemented everywhere.

Bootstrapped and regression tested on aarch64-linux-gnu, and committed
to mainline.

Regards,
Iain.

---
gcc/d/ChangeLog:

	PR d/103520
	* d-frontend.h (gc_disable): Declare.
	* d-lang.cc (d_init_options): Disable the D runtime garbage collector
	after initializing.
---
 gcc/d/d-frontend.h | 2 +-
 gcc/d/d-lang.cc    | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
diff mbox series

Patch

diff --git a/gcc/d/d-frontend.h b/gcc/d/d-frontend.h
index 3edf812212a..e7695d5718b 100644
--- a/gcc/d/d-frontend.h
+++ b/gcc/d/d-frontend.h
@@ -21,7 +21,7 @@  along with GCC; see the file COPYING3.  If not see
 /* These functions are defined in D runtime.  */
 extern "C" int rt_init (void);
 extern "C" int rt_term (void);
-//extern "C" void gc_disable (void);
+extern "C" void gc_disable (void);
 extern "C" void *gc_malloc (size_t sz, unsigned ba = 0, const void *ti = NULL);
 extern "C" void gc_free (void *);
 extern "C" void gc_collect (void);
diff --git a/gcc/d/d-lang.cc b/gcc/d/d-lang.cc
index d20370e5d8a..dbf7a8b60b9 100644
--- a/gcc/d/d-lang.cc
+++ b/gcc/d/d-lang.cc
@@ -288,7 +288,7 @@  d_init_options (unsigned int, cl_decoded_option *decoded_options)
 {
   /* Initialize the D runtime.  */
   rt_init ();
-//  gc_disable ();
+  gc_disable ();
 
   /* Set default values.  */
   global._init ();