fixup! Some fixes and workaround for aarch64

Author: antoyo

gcc/config/aarch64/aarch64-builtins.cc

@@ -1274,8 +1274,6 @@ aarch64_init_simd_builtin_types (void)
 
       if (aarch64_simd_types_trees[i].itype == NULL)
 	{
-          // FIXME: problem happens when we execute this a second time.
-          // Currently fixed by not initializing builtins more than once in dummy-frontend.cc.
 	  tree type = build_vector_type (eltype, GET_MODE_NUNITS (mode));
 	  type = build_distinct_type_copy (type);
 	  SET_TYPE_STRUCTURAL_EQUALITY (type);

gcc/config/aarch64/aarch64.cc

@@ -16307,19 +16307,11 @@ aarch64_insn_cost (rtx_insn *insn, bool speed)
 static void
 aarch64_init_builtins ()
 {
-  // TODO: find a better fix than this to avoid the failure:
-  // SET_TYPE_VECTOR_SUBPARTS, in tree.h:4258
-  static bool builtins_initialized = false;
-  if (builtins_initialized)
-    return;
-
   aarch64_general_init_builtins ();
   aarch64_sve::init_builtins ();
 #ifdef SUBTARGET_INIT_BUILTINS
   SUBTARGET_INIT_BUILTINS;
 #endif
-
-  builtins_initialized = true;
 }
 
 /* Implement TARGET_FOLD_BUILTIN.  */

gcc/jit/dummy-frontend.cc

@@ -1124,16 +1124,7 @@ jit_langhook_init (void)
      eventually be controllable by a command line option.  */
   mpfr_set_default_prec (256);
 
-  // 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;
 }