Merge pull request #54 from GuillaumeGomez/target-builtins

Fix wrong reinitialization of `target_builtins`

Author: antoyo

gcc/jit/dummy-frontend.cc

@@ -1035,8 +1035,6 @@ jit_end_diagnostic (diagnostic_context *context,
   gcc::jit::active_playback_ctxt->add_diagnostic (context, *diagnostic);
 }
 
-static bool builtins_initialized = false;
-
 /* Language hooks.  */
 
 static bool
@@ -1075,12 +1073,16 @@ jit_langhook_init (void)
      eventually be controllable by a command line option.  */
   mpfr_set_default_prec (256);
 
-  // TODO: check if this is a good fix.
-  if (!builtins_initialized)
-  {
-    targetm.init_builtins ();
-    builtins_initialized = true;
-  }
+  // FIXME: This code doesn't work as it erases the `target_builtins` map
+  // without checking if it's already filled before. A better check would be
+  // `if target_builtins.len() == 0` (or whatever this `hash_map` type method
+  // name is).
+  // static bool builtins_initialized = false;
+  // if (!builtins_initialized)
+  // {
+  targetm.init_builtins ();
+  //   builtins_initialized = true;
+  // }
 
   return true;
 }