Merge pull request #40561 from kubamracek/avoid-building-runtime-reflection-code-when-reflection-is-off

When SWIFT_ENABLE_REFLECTION is Off, don't build the reflection runtime code (stdlib/public/Reflection/*.cpp)

Author: kubamracek

lib/SwiftReflection/CMakeLists.txt

@@ -4,5 +4,6 @@ add_swift_host_library(swiftReflection STATIC
   ${SWIFT_SOURCE_DIR}/stdlib/public/Reflection/TypeLowering.cpp
   ${SWIFT_SOURCE_DIR}/stdlib/public/Reflection/TypeRef.cpp
   ${SWIFT_SOURCE_DIR}/stdlib/public/Reflection/TypeRefBuilder.cpp)
+target_compile_options(swiftReflection PRIVATE -DSWIFT_ENABLE_REFLECTION)
 target_link_libraries(swiftReflection PUBLIC
   swiftDemangling)

stdlib/public/Reflection/CMakeLists.txt

@@ -4,9 +4,14 @@ set(swiftReflection_SOURCES
   TypeRef.cpp
   TypeRefBuilder.cpp)
 
+set(swiftReflection_C_COMPILE_FLAGS -DswiftCore_EXPORTS)
+if(SWIFT_ENABLE_REFLECTION)
+  list(APPEND swiftReflection_C_COMPILE_FLAGS -DSWIFT_ENABLE_REFLECTION)
+endif()
+
 add_swift_target_library(swiftReflection STATIC
   ${swiftReflection_SOURCES}
-  C_COMPILE_FLAGS ${SWIFT_RUNTIME_CXX_FLAGS} -DswiftCore_EXPORTS
+  C_COMPILE_FLAGS ${SWIFT_RUNTIME_CXX_FLAGS} ${swiftReflection_C_COMPILE_FLAGS}
   LINK_FLAGS ${SWIFT_RUNTIME_LINK_FLAGS}
   INCORPORATE_OBJECT_LIBRARIES
     swiftLLVMSupport swiftDemangling

stdlib/public/Reflection/MetadataSource.cpp

@@ -10,6 +10,8 @@
 //
 //===----------------------------------------------------------------------===//
 
+#if SWIFT_ENABLE_REFLECTION
+
 #include "swift/Reflection/MetadataSource.h"
 #include <iostream>
 
@@ -103,3 +105,5 @@ void MetadataSource::dump(std::ostream &stream, unsigned Indent) const {
   PrintMetadataSource(stream, Indent).visit(this);
   stream << "\n";
 }
+
+#endif

stdlib/public/Reflection/TypeLowering.cpp

@@ -18,6 +18,8 @@
 //
 //===----------------------------------------------------------------------===//
 
+#if SWIFT_ENABLE_REFLECTION
+
 #include "swift/ABI/Enum.h"
 #include "swift/ABI/MetadataValues.h"
 #include "swift/Reflection/TypeLowering.h"
@@ -2300,3 +2302,5 @@ const TypeInfo *TypeConverter::getClassInstanceTypeInfo(
 
 } // namespace reflection
 } // namespace swift
+
+#endif

stdlib/public/Reflection/TypeRef.cpp

@@ -15,6 +15,8 @@
 //
 //===----------------------------------------------------------------------===//
 
+#if SWIFT_ENABLE_REFLECTION
+
 #include "swift/Basic/Range.h"
 #include "swift/Demangling/Demangle.h"
 #include "swift/Reflection/TypeRef.h"
@@ -1478,3 +1480,5 @@ bool TypeRef::deriveSubstitutions(GenericArgumentMap &Subs,
   // exactly.
   return (OrigTR == SubstTR);
 }
+
+#endif

stdlib/public/Reflection/TypeRefBuilder.cpp

@@ -15,6 +15,8 @@
 //
 //===----------------------------------------------------------------------===//
 
+#if SWIFT_ENABLE_REFLECTION
+
 #include "swift/Reflection/TypeRefBuilder.h"
 
 #include "swift/Demangling/Demangle.h"
@@ -510,3 +512,5 @@ void TypeRefBuilder::dumpAllSections(std::ostream &stream) {
   dumpCaptureSection(stream);
   stream << "\n";
 }
+
+#endif

stdlib/public/runtime/Demangle.cpp

@@ -13,7 +13,6 @@
 #include "Private.h"
 #include "swift/ABI/TypeIdentity.h"
 #include "swift/Basic/Range.h"
-#include "swift/Reflection/TypeRef.h"
 #include "swift/Runtime/Metadata.h"
 #include "swift/Runtime/Portability.h"
 #include "swift/Strings.h"