[cling] Restrict BuiltinHeadersInSystemModules to non-Apple

hahnjo · hahnjo · commit c4fdf33661db · 2026-06-16T11:13:41.000+02:00
It was introduced in commit c28b205 during the upgrade to LLVM 18 to fix the build on (basically) all platforms, and this remains true on Linux. For Apple however, the newest MacOSX27.0.sdk is incomaptible and the option must not be set.
diff --git a/interpreter/cling/lib/Interpreter/CIFactory.cpp b/interpreter/cling/lib/Interpreter/CIFactory.cpp
@@ -434,7 +434,9 @@ namespace {
     }
 
     //Opts.Modules = 1;
+#ifndef __APPLE__
     Opts.BuiltinHeadersInSystemModules = 1;
+#endif
 
     // See test/CodeUnloading/PCH/VTables.cpp which implicitly compares clang
     // to cling lang options. They should be the same, we should not have to

Original file line number	Diff line number	Diff line change
`@@ -434,7 +434,9 @@ namespace {`
`434`	`434`	`}`
`435`	`435`
`436`	`436`	`//Opts.Modules = 1;`
	`437`	`+#ifndef __APPLE__`
`437`	`438`	`Opts.BuiltinHeadersInSystemModules = 1;`
	`439`	`+#endif`
`438`	`440`
`439`	`441`	`// See test/CodeUnloading/PCH/VTables.cpp which implicitly compares clang`
`440`	`442`	`// to cling lang options. They should be the same, we should not have to`