Skip to content

Commit 84a1ffc

Browse files
committed
[Shims] Include SwiftShims headers without ../
This replaces a number of `#include`-s like this: ``` #include "../../../stdlib/public/SwiftShims/Visibility.h" ``` with this: ``` #include "swift/shims/Visibility.h" ``` This is needed to allow SwiftCompilerSources to use C++ headers which include SwiftShims headers. Currently trying to do that results in errors: ``` swift/swift/include/swift/Demangling/../../../stdlib/public/SwiftShims/module.modulemap:1:8: error: redefinition of module 'SwiftShims' module SwiftShims { ^ Builds.noindex/swift/swift/bootstrapping0/lib/swift/shims/module.modulemap:1:8: note: previously defined here module SwiftShims { ^ ``` This happens because the headers in both the source dir and the build dir refer to SwiftShims headers by relative path, and both the source root and the build root contain SwiftShims headers (which are equivalent, but since they are located in different dirs, Clang treats them as different modules).
1 parent 435af14 commit 84a1ffc

File tree

99 files changed

+84
-82
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

99 files changed

+84
-82
lines changed

CMakeLists.txt

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -735,6 +735,7 @@ get_filename_component(SWIFT_SOURCE_DIR ${CMAKE_CURRENT_SOURCE_DIR} REALPATH)
735735
set(SWIFT_BINARY_DIR "${CMAKE_CURRENT_BINARY_DIR}")
736736
set(SWIFT_CMAKE_DIR "${CMAKE_CURRENT_SOURCE_DIR}/cmake/modules")
737737
set(SWIFT_MAIN_INCLUDE_DIR "${SWIFT_SOURCE_DIR}/include")
738+
set(SWIFT_SHIMS_INCLUDE_DIR "${SWIFT_SOURCE_DIR}/stdlib/public/SwiftShims")
738739
set(SWIFT_INCLUDE_DIR "${CMAKE_CURRENT_BINARY_DIR}/include")
739740

740741
set(SWIFT_RUNTIME_OUTPUT_INTDIR "${CMAKE_BINARY_DIR}/${CMAKE_CFG_INTDIR}/bin")
@@ -785,6 +786,7 @@ set(CMAKE_INCLUDE_CURRENT_DIR ON)
785786
include_directories(BEFORE
786787
${SWIFT_MAIN_INCLUDE_DIR}
787788
${SWIFT_INCLUDE_DIR}
789+
${SWIFT_SHIMS_INCLUDE_DIR}
788790
)
789791

790792
# Configuration flags passed to all of our invocations of gyb. Try to

cmake/modules/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ set(SWIFT_CONFIG_EXPORTS ${SWIFT_EXPORTS} ${SWIFT_BUILDTREE_EXPORTS})
1010
export(TARGETS ${SWIFT_CONFIG_EXPORTS} FILE ${SWIFT_EXPORTS_FILE})
1111

1212

13-
set(SWIFT_INCLUDE_DIRS ${SWIFT_INCLUDE_DIR} ${SWIFT_MAIN_INCLUDE_DIR})
13+
set(SWIFT_INCLUDE_DIRS ${SWIFT_INCLUDE_DIR} ${SWIFT_MAIN_INCLUDE_DIR} ${SWIFT_SHIMS_INCLUDE_DIR})
1414
set(SWIFT_LIBRARY_DIRS ${SWIFT_LIBRARY_OUTPUT_INTDIR})
1515

1616
configure_file(

include/swift/ABI/HeapObject.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,6 @@
1717
#ifndef SWIFT_ABI_HEAPOBJECT_H
1818
#define SWIFT_ABI_HEAPOBJECT_H
1919

20-
#include "../../../stdlib/public/SwiftShims/HeapObject.h"
20+
#include "swift/shims/HeapObject.h"
2121

2222
#endif // SWIFT_ABI_HEAPOBJECT_H

include/swift/ABI/KeyPath.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222

2323
#include <cstdint>
2424
#include <cassert>
25-
#include "../../../stdlib/public/SwiftShims/KeyPath.h"
25+
#include "swift/shims/KeyPath.h"
2626

2727
namespace swift {
2828

include/swift/ABI/Metadata.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@
4040
#include "swift/Demangling/Demangle.h"
4141
#include "swift/Demangling/ManglingMacros.h"
4242
#include "swift/Basic/Unreachable.h"
43-
#include "../../../stdlib/public/SwiftShims/HeapObject.h"
43+
#include "swift/shims/HeapObject.h"
4444
#include "llvm/ADT/STLExtras.h"
4545
#include "llvm/Support/Casting.h"
4646

include/swift/ABI/System.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,6 @@
1717
#ifndef SWIFT_ABI_SYSTEM_H
1818
#define SWIFT_ABI_SYSTEM_H
1919

20-
#include "../../../stdlib/public/SwiftShims/System.h"
20+
#include "swift/shims/System.h"
2121

2222
#endif // SWIFT_ABI_SYSTEM_H

include/swift/Demangling/Errors.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,8 +19,8 @@
1919
#ifndef SWIFT_DEMANGLING_ERRORS_H
2020
#define SWIFT_DEMANGLING_ERRORS_H
2121

22-
#include "../../../stdlib/public/SwiftShims/Visibility.h"
2322
#include "swift/Demangling/NamespaceMacros.h"
23+
#include "swift/shims/Visibility.h"
2424
#include <inttypes.h>
2525
#include <stdarg.h>
2626

include/swift/Runtime/Config.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -166,7 +166,7 @@ extern uintptr_t __COMPATIBILITY_LIBRARIES_CANNOT_CHECK_THE_IS_SWIFT_BIT_DIRECTL
166166
#endif
167167

168168
// Bring in visibility attribute macros
169-
#include "../../../stdlib/public/SwiftShims/Visibility.h"
169+
#include "swift/shims/Visibility.h"
170170

171171
// Define mappings for calling conventions.
172172

include/swift/Runtime/Heap.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222
#include <utility>
2323

2424
#include "swift/Runtime/Config.h"
25-
#include "../../../stdlib/public/SwiftShims/Visibility.h"
25+
#include "swift/shims/Visibility.h"
2626

2727
namespace swift {
2828
// Allocate plain old memory. This is the generalized entry point

include/swift/Runtime/HeapObject.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,8 @@
2727
#endif // SWIFT_OBJC_INTEROP
2828

2929
// Bring in the definition of HeapObject
30-
#include "../../../stdlib/public/SwiftShims/HeapObject.h"
31-
#include "../../../stdlib/public/SwiftShims/Visibility.h"
30+
#include "swift/shims/HeapObject.h"
31+
#include "swift/shims/Visibility.h"
3232

3333
namespace swift {
3434

include/swift/Runtime/Metadata.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020
#include "swift/ABI/Metadata.h"
2121
#include "swift/Reflection/Records.h"
2222
#include "swift/Runtime/Once.h"
23-
#include "../../../stdlib/public/SwiftShims/Visibility.h"
23+
#include "swift/shims/Visibility.h"
2424

2525
namespace swift {
2626

lib/Demangling/CrashReporter.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222
#if SWIFT_HAVE_CRASHREPORTERCLIENT
2323

2424
// For SWIFT_LIBRARY_VISIBILITY
25-
#include "../../stdlib/public/SwiftShims/Visibility.h"
25+
#include "swift/shims/Visibility.h"
2626

2727
#include <inttypes.h>
2828

stdlib/linker-support/magic-symbols-for-install-name.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121

2222
#include <Availability.h>
2323
#include <TargetConditionals.h>
24-
#include "../public/SwiftShims/Visibility.h"
24+
#include "swift/shims/Visibility.h"
2525

2626
#define RPATH_INSTALL_NAME_DIRECTIVE_IMPL2(name, major, minor) \
2727
SWIFT_RUNTIME_EXPORT const char install_name_ ## major ## _ ## minor \

stdlib/public/BackDeployConcurrency/Error.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919

2020
#include "swift/Basic/Compiler.h"
2121

22-
#include "../SwiftShims/Visibility.h"
22+
#include "swift/shims/Visibility.h"
2323
#include <cstdarg>
2424
#include <cstdint>
2525
#include <cstdlib>

stdlib/public/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ if(CXX_SUPPORTS_EXIT_TIME_DESTRUCTORS_WARNING)
6262
"-Werror=exit-time-destructors")
6363
endif()
6464

65-
add_subdirectory(SwiftShims)
65+
add_subdirectory(SwiftShims/swift/shims)
6666
add_subdirectory(CommandLineSupport)
6767
add_subdirectory(Threading)
6868

stdlib/public/CommandLineSupport/CommandLine.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,9 +26,9 @@
2626

2727
#include "swift/Runtime/Debug.h"
2828

29-
#include "../SwiftShims/GlobalObjects.h"
30-
#include "../SwiftShims/RuntimeStubs.h"
31-
#include "../SwiftShims/Visibility.h"
29+
#include "swift/shims/GlobalObjects.h"
30+
#include "swift/shims/RuntimeStubs.h"
31+
#include "swift/shims/Visibility.h"
3232

3333
#if defined(_WIN32)
3434
#define WIN32_LEAN_AND_MEAN

stdlib/public/Concurrency/Error.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919

2020
#include "swift/Basic/Compiler.h"
2121

22-
#include "../SwiftShims/Visibility.h"
22+
#include "swift/shims/Visibility.h"
2323
#include <cstdarg>
2424
#include <cstdint>
2525
#include <cstdlib>

stdlib/public/Concurrency/linker-support/magic-symbols-for-install-name.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@
2424

2525
#include <Availability.h>
2626
#include <TargetConditionals.h>
27-
#include "../../SwiftShims/Visibility.h"
27+
#include "swift/shims/Visibility.h"
2828

2929
#define RPATH_INSTALL_NAME_DIRECTIVE_IMPL2(name, major, minor) \
3030
SWIFT_RUNTIME_EXPORT const char install_name_ ## major ## _ ## minor \

stdlib/public/runtime/AnyHashableSupport.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@
1010
//
1111
//===----------------------------------------------------------------------===//
1212

13-
#include "../SwiftShims/Visibility.h"
1413
#include "Private.h"
1514
#include "SwiftHashableSupport.h"
1615
#include "SwiftValue.h"
@@ -20,6 +19,7 @@
2019
#include "swift/Runtime/Config.h"
2120
#include "swift/Runtime/Debug.h"
2221
#include "swift/Runtime/HeapObject.h"
22+
#include "swift/shims/Visibility.h"
2323

2424
#include <new>
2525

stdlib/public/runtime/Bincompat.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919
#include "swift/Runtime/Debug.h"
2020
#include "swift/Runtime/EnvironmentVariables.h"
2121
#include "swift/Threading/Once.h"
22-
#include "../SwiftShims/RuntimeShims.h"
22+
#include "swift/shims/RuntimeShims.h"
2323
#include <stdint.h>
2424

2525
// If this is an Apple OS, use the Apple binary compatibility rules

stdlib/public/runtime/Casting.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,6 @@
1717

1818
#include "swift/Runtime/Casting.h"
1919
#include "../CompatibilityOverride/CompatibilityOverride.h"
20-
#include "../SwiftShims/GlobalObjects.h"
21-
#include "../SwiftShims/RuntimeShims.h"
2220
#include "ErrorObject.h"
2321
#include "ExistentialMetadataImpl.h"
2422
#include "Private.h"
@@ -32,6 +30,8 @@
3230
#include "swift/Runtime/ExistentialContainer.h"
3331
#include "swift/Runtime/HeapObject.h"
3432
#include "swift/Runtime/Metadata.h"
33+
#include "swift/shims/GlobalObjects.h"
34+
#include "swift/shims/RuntimeShims.h"
3535
#include "swift/Threading/Mutex.h"
3636
#include "llvm/ADT/DenseMap.h"
3737
#include "llvm/ADT/PointerIntPair.h"

stdlib/public/runtime/Exclusivity.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@
2424
#endif
2525

2626
#include "swift/Runtime/Exclusivity.h"
27-
#include "../SwiftShims/Visibility.h"
27+
#include "swift/shims/Visibility.h"
2828
#include "SwiftTLSContext.h"
2929
#include "swift/Basic/Lazy.h"
3030
#include "swift/Runtime/Config.h"

stdlib/public/runtime/Float16Support.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@
3232
#if (defined(__ANDROID__) && defined(__ARM_ARCH_7A__) && defined(__ARM_EABI__)) || \
3333
((defined(__i386__) || defined(__i686__) || defined(__x86_64__)) && !defined(__APPLE__))
3434

35-
#include "../SwiftShims/Visibility.h"
35+
#include "swift/shims/Visibility.h"
3636

3737
static unsigned toEncoding(float f) {
3838
unsigned e;

stdlib/public/runtime/Heap.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
#include "swift/Runtime/Heap.h"
1919
#include "Private.h"
2020
#include "swift/Runtime/Debug.h"
21-
#include "../SwiftShims/RuntimeShims.h"
21+
#include "swift/shims/RuntimeShims.h"
2222
#include <algorithm>
2323
#include <stdlib.h>
2424
#if defined(__APPLE__) && SWIFT_STDLIB_HAS_DARWIN_LIBMALLOC

stdlib/public/runtime/HeapObject.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,15 +26,15 @@
2626
#include "WeakReference.h"
2727
#include "swift/Runtime/Debug.h"
2828
#include "swift/Runtime/InstrumentsSupport.h"
29+
#include "swift/shims/GlobalObjects.h"
30+
#include "swift/shims/RuntimeShims.h"
2931
#include <algorithm>
3032
#include <cassert>
3133
#include <cstring>
3234
#include <cstdio>
3335
#include <cstdlib>
3436
#include <new>
3537
#include <thread>
36-
#include "../SwiftShims/GlobalObjects.h"
37-
#include "../SwiftShims/RuntimeShims.h"
3838
#if SWIFT_OBJC_INTEROP
3939
# include <objc/NSObject.h>
4040
# include <objc/runtime.h>

stdlib/public/runtime/ImageInspectionCommon.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,8 +21,8 @@
2121

2222
#if !defined(__MACH__)
2323

24-
#include "../SwiftShims/Visibility.h"
25-
#include "../SwiftShims/MetadataSections.h"
24+
#include "swift/shims/Visibility.h"
25+
#include "swift/shims/MetadataSections.h"
2626
#include "ImageInspection.h"
2727
#include "swift/Basic/Lazy.h"
2828
#include "swift/Runtime/Concurrent.h"

stdlib/public/runtime/ImageInspectionCommon.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@
4646
#define SWIFT_REFLECTION_METADATA_ELF_NOTE_MAGIC_STRING "swift_reflection_metadata_magic_string"
4747
#endif // defined(__ELF__)
4848

49-
#include "../SwiftShims/Visibility.h"
49+
#include "swift/shims/Visibility.h"
5050
#include <cstdint>
5151
#include <cstddef>
5252

stdlib/public/runtime/ImageInspectionELF.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020

2121
#if defined(__ELF__)
2222

23-
#include "../SwiftShims/MetadataSections.h"
23+
#include "swift/shims/MetadataSections.h"
2424
#include "ImageInspection.h"
2525
#include <dlfcn.h>
2626

stdlib/public/runtime/ImageInspectionWasm.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818

1919
#if defined(__wasm__)
2020

21-
#include "../SwiftShims/MetadataSections.h"
21+
#include "swift/shims/MetadataSections.h"
2222
#include "ImageInspection.h"
2323

2424
using namespace swift;

stdlib/public/runtime/Leaks.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121

2222
#if SWIFT_RUNTIME_ENABLE_LEAK_CHECKER
2323

24-
#include "../SwiftShims/Visibility.h"
24+
#include "swift/shims/Visibility.h"
2525

2626
#include "swift/Runtime/Config.h"
2727

stdlib/public/runtime/MetadataCache.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
#include "swift/Runtime/Metadata.h"
1818
#include "swift/Threading/Mutex.h"
1919

20-
#include "../SwiftShims/Visibility.h"
20+
#include "swift/shims/Visibility.h"
2121

2222
#include "llvm/ADT/Hashing.h"
2323
#include "llvm/ADT/STLExtras.h"

stdlib/public/runtime/Private.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323
#include "swift/Demangling/TypeLookupError.h"
2424
#include "swift/Runtime/Config.h"
2525
#include "swift/Runtime/Metadata.h"
26-
#include "../SwiftShims/Visibility.h"
26+
#include "swift/shims/Visibility.h"
2727

2828
#if defined(__APPLE__) && __has_include(<TargetConditionals.h>)
2929
#include <TargetConditionals.h>

stdlib/public/runtime/ReflectionMirror.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@
1212

1313
#ifdef SWIFT_ENABLE_REFLECTION
1414

15-
#include "../SwiftShims/Reflection.h"
1615
#include "ImageInspection.h"
1716
#include "Private.h"
1817
#include "WeakReference.h"
@@ -27,6 +26,7 @@
2726
#include "swift/Runtime/Metadata.h"
2827
#include "swift/Runtime/Portability.h"
2928
#include "swift/Runtime/Reflection.h"
29+
#include "swift/shims/Reflection.h"
3030
#include <cassert>
3131
#include <cinttypes>
3232
#include <cstdio>

stdlib/public/runtime/SwiftObject.mm

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,8 +34,8 @@
3434
#include "swift/Runtime/ObjCBridge.h"
3535
#include "swift/Runtime/Portability.h"
3636
#include "swift/Strings.h"
37-
#include "../SwiftShims/RuntimeShims.h"
38-
#include "../SwiftShims/AssertionReporting.h"
37+
#include "swift/shims/RuntimeShims.h"
38+
#include "swift/shims/AssertionReporting.h"
3939
#include "../CompatibilityOverride/CompatibilityOverride.h"
4040
#include "ErrorObject.h"
4141
#include "Private.h"

stdlib/public/runtime/SwiftRT-COFF.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
//===----------------------------------------------------------------------===//
1212

1313
#include "ImageInspectionCommon.h"
14-
#include "../SwiftShims/MetadataSections.h"
14+
#include "swift/shims/MetadataSections.h"
1515

1616
#include <cstdint>
1717
#include <new>

stdlib/public/runtime/SwiftRT-ELF.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
//===----------------------------------------------------------------------===//
1212

1313
#include "ImageInspectionCommon.h"
14-
#include "../SwiftShims/MetadataSections.h"
14+
#include "swift/shims/MetadataSections.h"
1515

1616
#include <cstddef>
1717
#include <new>

stdlib/public/runtime/WeakReference.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,8 @@
1717
#ifndef SWIFT_RUNTIME_WEAKREFERENCE_H
1818
#define SWIFT_RUNTIME_WEAKREFERENCE_H
1919

20-
#include "../../../stdlib/public/SwiftShims/Target.h"
21-
#include "../SwiftShims/Visibility.h"
20+
#include "swift/shims/Target.h"
21+
#include "swift/shims/Visibility.h"
2222
#include "swift/Runtime/Config.h"
2323
#include "swift/Runtime/HeapObject.h"
2424
#include "swift/Runtime/Metadata.h"

stdlib/public/stubs/Assert.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
#include "swift/Runtime/Config.h"
1414
#include "swift/Runtime/Debug.h"
1515
#include "swift/Runtime/Portability.h"
16-
#include "SwiftShims/AssertionReporting.h"
16+
#include "swift/shims/AssertionReporting.h"
1717
#include <cstdarg>
1818
#include <cstdint>
1919
#include <stdio.h>

0 commit comments

Comments
 (0)