Remove the need to preload exports

Author: josevalim

lib/elixir/lib/module/parallel_checker.ex

@@ -185,27 +185,19 @@ defmodule Module.ParallelChecker do
     {checker, :gen_server.call(checker, :ets, :infinity)}
   end
 
-  @doc """
-  Preloads a module into the cache. Call this function before any other
-  cache lookups for the module.
-  """
-  @spec preload_module(cache(), module()) :: :ok
-  def preload_module({server, ets}, module) do
-    case :ets.lookup(ets, {:cached, module}) do
-      [{_key, _}] -> :ok
-      [] -> cache_module({server, ets}, module)
-    end
-  end
-
   @doc """
   Returns the export kind and deprecation reason for the given MFA from
   the cache. If the module does not exist return `{:error, :module}`,
   or if the function does not exist return `{:error, :function}`.
   """
   @spec fetch_export(cache(), module(), atom(), arity()) ::
           {:ok, mode(), kind(), binary() | nil} | {:error, :function | :module}
-  def fetch_export({_server, ets}, module, fun, arity) do
+  def fetch_export({server, ets}, module, fun, arity) do
     case :ets.lookup(ets, {:cached, module}) do
+      [] ->
+        cache_module({server, ets}, module)
+        fetch_export({server, ets}, module, fun, arity)
+
       [{_key, false}] ->
         {:error, :module}
 

lib/elixir/lib/module/types/of.ex

@@ -431,7 +431,6 @@ defmodule Module.Types.Of do
     if Keyword.get(meta, :runtime_module, false) do
       context
     else
-      ParallelChecker.preload_module(stack.cache, module)
       check_export(module, fun, arity, meta, stack, context)
     end
   end