Don’t include SourceLoc.h when USED_IN_CPP_SOURCE is not set

Author: ahoppen

include/swift/Basic/BasicBridging.h

@@ -22,16 +22,15 @@
 //
 #include "swift/Basic/BridgedSwiftObject.h"
 #include "swift/Basic/Compiler.h"
-#include "swift/Basic/SourceLoc.h"
-// Workaround to avoid a compiler error because `cas::ObjectRef` is not defined
-// when including VirtualFileSystem.h
-#include "llvm/CAS/CASReference.h"
 
 #include <stddef.h>
 #include <stdint.h>
 #include <vector>
 #ifdef USED_IN_CPP_SOURCE
+// Workaround to avoid a compiler error because `cas::ObjectRef` is not defined
+// when including VirtualFileSystem.h
 #include <cassert>
+#include "llvm/CAS/CASReference.h"
 
 #include "swift/Basic/SourceLoc.h"
 #include "llvm/ADT/StringRef.h"

include/swift/SILOptimizer/OptimizerBridging.h

@@ -24,6 +24,17 @@
 
 #include "swift/SILOptimizer/Analysis/BasicCalleeAnalysis.h"
 
+#else // USED_IN_CPP_SOURCE
+
+#error "should not include swift headers into bridging header"
+
+#ifdef SWIFT_SIL_SILVALUE_H
+#error "should not include swift headers into bridging header"
+#endif
+#ifdef LLVM_SUPPORT_COMPILER_H
+#error "should not include llvm headers into bridging header"
+#endif
+
 #endif // USED_IN_CPP_SOURCE
 
 SWIFT_BEGIN_NULLABILITY_ANNOTATIONS