[clang] [llvm] [clang][modules] Separate parsing of modulemaps (PR #119740)

[Jakub Kuderski via llvm-commits]

================
@@ -2596,6 +2596,15 @@ template <typename T> using has_sizeof = decltype(sizeof(T));
 template <typename T>
 constexpr bool is_incomplete_v = !is_detected<detail::has_sizeof, T>::value;
 
+//===----------------------------------------------------------------------===//
+//     Extra additions to <variant>
+//===----------------------------------------------------------------------===//
+
+template <class... Ts> struct overloaded : Ts... {
+  using Ts::operator()...;
+};
+template <class... Ts> overloaded(Ts...) -> overloaded<Ts...>;
+
----------------
kuhar wrote:

Can you put this in a separate PR? This way we won't have to rebuild the whole tree if this larger PR gets reverted.

Also, doesn't this duplicate `llvm::make_visitor`? Maybe we could merge the two.

https://github.com/llvm/llvm-project/pull/119740