Add a CLI flag for incremental GC

include/cpp/jank/util/cli.hpp

@@ -17,6 +17,7 @@ namespace jank::util::cli
     native_string class_path;
     native_bool profiler_enabled{};
     native_string profiler_file{ "jank.profile" };
+    native_bool gc_incremental{};
 
     /* Compilation. */
     native_string compilation_path{ "classes" };

src/cpp/jank/util/cli.cpp

@@ -14,6 +14,7 @@ namespace jank::util::cli
     cli.add_option("--class-path", opts.class_path, fmt::format("A {} separated list of directories, JAR files, and ZIP files to search for modules", runtime::module::loader::module_separator));
     cli.add_flag("--profile", opts.profiler_enabled, "Enable compiler and runtime profiling");
     cli.add_option("--profile-output", opts.profiler_file, "The file to write profile entries (will be overwritten)");
+    cli.add_flag("--gc-incremental", opts.gc_incremental, "Enable incremental GC collection");
 
     /* Compilation. */
     cli.add_option("--output-dir", opts.compilation_path, "The base directory where compiled modules are written");
@@ -41,7 +42,7 @@ namespace jank::util::cli
     {
       cli.parse(argc, argv);
     }
-    catch (CLI::ParseError const &e)
+    catch(CLI::ParseError const &e)
     { return err(cli.exit(e)); }
 
     if(cli.got_subcommand(&cli_run))

src/cpp/main.cpp

@@ -118,10 +118,14 @@ try
    * like strings, use the GC for allocations. It can still be configured later. */
   GC_set_all_interior_pointers(1);
   GC_enable();
-  /* TODO: This crashes now, with LLVM13. Looks like it's cleaning up things it shouldn't. */
-  //GC_enable_incremental();
 
-  auto const opts(util::cli::parse(argc, argv).expect_ok());
+  auto const parse_result(util::cli::parse(argc, argv));
+  if(parse_result.is_err())
+  { return parse_result.expect_err(); }
+  auto const &opts(parse_result.expect_ok());
+
+  if(opts.gc_incremental)
+  { GC_enable_incremental(); }
 
   profile::configure(opts);
   profile::timer timer{ "main" };