Skip to content

Commit 1fb9249

Browse files
tinysun212tinysun212
committed
[swiftc] Fixed for Cygwin
Fixed for the difference of Cygwin with other Windows variants (MSVC, Itanium, MinGW). - The platform name is renamed to "cygwin" from "windows" which is used for searching the standard libraries. - The consideration for DLL storage class (DllExport/DllImport) is not required for Cygwin and MinGW. There is no problem when linking in these environment. - Cygwin should use large memory model as default.(This may be changed if someone ports to 32bit) - Cygwin and MinGW should use the autolink feature in the sameway of Linux due to the linker's limit.
1 parent 13a9908 commit 1fb9249

File tree

13 files changed

+55
-33
lines changed

13 files changed

+55
-33
lines changed

‎CMakeLists.txt‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -610,10 +610,10 @@ elseif("${SWIFT_HOST_VARIANT_SDK}" STREQUAL "FREEBSD")
610610
elseif("${SWIFT_HOST_VARIANT_SDK}" STREQUAL "CYGWIN")
611611

612612
# set(CMAKE_EXECUTABLE_FORMAT "ELF")
613-
set(SWIFT_HOST_VARIANT "windows" CACHE STRING
614-
"Deployment OS for Swift host tools (the compiler) [windows].")
613+
set(SWIFT_HOST_VARIANT "cygwin" CACHE STRING
614+
"Deployment OS for Swift host tools (the compiler) [cygwin].")
615615

616-
configure_sdk_unix(CYGWIN "Cygwin" "windows" "cygwin" "windows" "x86_64-unknown-windows-cygnus" "/")
616+
configure_sdk_unix(CYGWIN "Cygwin" "cygwin" "cygwin" "x86_64" "x86_64-unknown-windows-cygnus" "/")
617617
set(SWIFT_PRIMARY_VARIANT_SDK_default "${SWIFT_HOST_VARIANT_SDK}")
618618
set(SWIFT_PRIMARY_VARIANT_ARCH_default "x86_64")
619619

‎include/swift/Runtime/Concurrent.h‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515
#include <atomic>
1616
#include <stdint.h>
1717

18-
#if defined(__FreeBSD__)
18+
#if defined(__FreeBSD__) || defined(__CYGWIN__)
1919
#include <stdio.h>
2020
#endif
2121

‎lib/Basic/Platform.cpp‎

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -118,7 +118,16 @@ StringRef swift::getPlatformNameForTriple(const llvm::Triple &triple) {
118118
case llvm::Triple::FreeBSD:
119119
return "freebsd";
120120
case llvm::Triple::Win32:
121-
return "windows";
121+
switch (triple.getEnvironment()) {
122+
case llvm::Triple::Cygnus:
123+
return "cygwin";
124+
case llvm::Triple::GNU:
125+
return "mingw";
126+
case llvm::Triple::MSVC:
127+
return "windows";
128+
default:
129+
llvm_unreachable("unsupported Windows environment");
130+
}
122131
case llvm::Triple::PS4:
123132
return "ps4";
124133
}

‎lib/IRGen/GenDecl.cpp‎

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -937,7 +937,7 @@ void IRGenModule::emitVTableStubs() {
937937
// For each eliminated method symbol create an alias to the stub.
938938
auto *alias = llvm::GlobalAlias::create(llvm::GlobalValue::ExternalLinkage,
939939
F.getName(), stub);
940-
if (Triple.isOSBinFormatCOFF())
940+
if (useDllStorage())
941941
alias->setDLLStorageClass(llvm::GlobalValue::DLLExportStorageClass);
942942
}
943943
}
@@ -1239,7 +1239,7 @@ getIRLinkage(IRGenModule &IGM, SILLinkage linkage, bool isFragile,
12391239

12401240
const auto ObjFormat = IGM.TargetInfo.OutputObjectFormat;
12411241
bool IsELFObject = ObjFormat == llvm::Triple::ELF;
1242-
bool IsCOFFObject = ObjFormat == llvm::Triple::COFF;
1242+
bool UseDLLStorage = IGM.useDllStorage();
12431243

12441244
// Use protected visibility for public symbols we define on ELF. ld.so
12451245
// doesn't support relative relocations at load time, which interferes with
@@ -1249,11 +1249,11 @@ getIRLinkage(IRGenModule &IGM, SILLinkage linkage, bool isFragile,
12491249
IsELFObject ? llvm::GlobalValue::ProtectedVisibility
12501250
: llvm::GlobalValue::DefaultVisibility;
12511251
llvm::GlobalValue::DLLStorageClassTypes ExportedStorage =
1252-
IsCOFFObject ? llvm::GlobalValue::DLLExportStorageClass
1253-
: llvm::GlobalValue::DefaultStorageClass;
1252+
UseDLLStorage ? llvm::GlobalValue::DLLExportStorageClass
1253+
: llvm::GlobalValue::DefaultStorageClass;
12541254
llvm::GlobalValue::DLLStorageClassTypes ImportedStorage =
1255-
IsCOFFObject ? llvm::GlobalValue::DLLImportStorageClass
1256-
: llvm::GlobalValue::DefaultStorageClass;
1255+
UseDLLStorage ? llvm::GlobalValue::DLLImportStorageClass
1256+
: llvm::GlobalValue::DefaultStorageClass;
12571257

12581258
if (isFragile) {
12591259
// Fragile functions/globals must be visible from outside, regardless of

‎lib/IRGen/GenFunc.cpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1499,7 +1499,7 @@ void irgen::emitBlockHeader(IRGenFunction &IGF,
14991499
auto NSConcreteStackBlock =
15001500
IGF.IGM.getModule()->getOrInsertGlobal("_NSConcreteStackBlock",
15011501
IGF.IGM.ObjCClassStructTy);
1502-
if (IGF.IGM.Triple.isOSBinFormatCOFF())
1502+
if (IGF.IGM.useDllStorage())
15031503
cast<llvm::GlobalVariable>(NSConcreteStackBlock)
15041504
->setDLLStorageClass(llvm::GlobalValue::DLLImportStorageClass);
15051505

‎lib/IRGen/GenMeta.cpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3982,7 +3982,7 @@ static void emitObjCClassSymbol(IRGenModule &IGM,
39823982
metadata->getLinkage(),
39833983
classSymbol.str(), metadata,
39843984
IGM.getModule());
3985-
if (IGM.TargetInfo.OutputObjectFormat == llvm::Triple::COFF)
3985+
if (IGM.useDllStorage())
39863986
alias->setDLLStorageClass(metadata->getDLLStorageClass());
39873987
}
39883988

‎lib/IRGen/IRGen.cpp‎

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -490,10 +490,13 @@ static createTargetMachine(IRGenOptions &Opts, ASTContext &Ctx) {
490490
}
491491

492492
// Create a target machine.
493-
llvm::TargetMachine *TargetMachine
494-
= Target->createTargetMachine(Triple.str(), CPU,
495-
targetFeatures, TargetOpts, Reloc::PIC_,
496-
CodeModel::Default, OptLevel);
493+
auto cmodel = CodeModel::Default;
494+
if (Triple.isWindowsCygwinEnvironment())
495+
cmodel = CodeModel::Large;
496+
497+
llvm::TargetMachine *TargetMachine =
498+
Target->createTargetMachine(Triple.str(), CPU, targetFeatures, TargetOpts,
499+
Reloc::PIC_, cmodel, OptLevel);
497500
if (!TargetMachine) {
498501
Ctx.Diags.diagnose(SourceLoc(), diag::no_llvm_target,
499502
Triple.str(), "no LLVM target machine");

‎lib/IRGen/IRGenModule.cpp‎

Lines changed: 14 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -446,6 +446,7 @@ llvm::Constant *swift::getRuntimeFn(llvm::Module &Module,
446446
fn->setCallingConv(cc);
447447

448448
if (llvm::Triple(Module.getTargetTriple()).isOSBinFormatCOFF() &&
449+
!llvm::Triple(Module.getTargetTriple()).isOSCygMing() &&
449450
(fn->getLinkage() == llvm::GlobalValue::ExternalLinkage ||
450451
fn->getLinkage() == llvm::GlobalValue::AvailableExternallyLinkage))
451452
fn->setDLLStorageClass(llvm::GlobalValue::DLLImportStorageClass);
@@ -525,7 +526,8 @@ llvm::Constant *swift::getWrapperFn(llvm::Module &Module,
525526
auto *globalFnPtr =
526527
new llvm::GlobalVariable(Module, fnPtrTy, false,
527528
llvm::GlobalValue::ExternalLinkage, 0, symbol);
528-
if (llvm::Triple(Module.getTargetTriple()).isOSBinFormatCOFF())
529+
if (llvm::Triple(Module.getTargetTriple()).isOSBinFormatCOFF() &&
530+
!llvm::Triple(Module.getTargetTriple()).isOSCygMing())
529531
globalFnPtr->setDLLStorageClass(llvm::GlobalValue::DLLImportStorageClass);
530532

531533
// Forward all arguments.
@@ -643,7 +645,7 @@ llvm::Constant *IRGenModule::getEmptyTupleMetadata() {
643645

644646
EmptyTupleMetadata =
645647
Module.getOrInsertGlobal("_TMT_", FullTypeMetadataStructTy);
646-
if (Triple.isOSBinFormatCOFF())
648+
if (useDllStorage())
647649
cast<llvm::GlobalVariable>(EmptyTupleMetadata)
648650
->setDLLStorageClass(llvm::GlobalValue::DLLImportStorageClass);
649651
return EmptyTupleMetadata;
@@ -657,7 +659,7 @@ llvm::Constant *IRGenModule::getObjCEmptyCachePtr() {
657659
// struct objc_cache _objc_empty_cache;
658660
ObjCEmptyCachePtr = Module.getOrInsertGlobal("_objc_empty_cache",
659661
OpaquePtrTy->getElementType());
660-
if (Triple.isOSBinFormatCOFF())
662+
if (useDllStorage())
661663
cast<llvm::GlobalVariable>(ObjCEmptyCachePtr)
662664
->setDLLStorageClass(llvm::GlobalValue::DLLImportStorageClass);
663665
} else {
@@ -690,7 +692,7 @@ Address IRGenModule::getAddrOfObjCISAMask() {
690692
assert(TargetInfo.hasISAMasking());
691693
if (!ObjCISAMaskPtr) {
692694
ObjCISAMaskPtr = Module.getOrInsertGlobal("swift_isaMask", IntPtrTy);
693-
if (Triple.isOSBinFormatCOFF())
695+
if (useDllStorage())
694696
cast<llvm::GlobalVariable>(ObjCISAMaskPtr)
695697
->setDLLStorageClass(llvm::GlobalValue::DLLImportStorageClass);
696698
}
@@ -856,7 +858,7 @@ void IRGenModule::addLinkLibrary(const LinkLibrary &linkLib) {
856858
llvm::SmallString<64> buf;
857859
encodeForceLoadSymbolName(buf, linkLib.getName());
858860
auto symbolAddr = Module.getOrInsertGlobal(buf.str(), Int1Ty);
859-
if (Triple.isOSBinFormatCOFF())
861+
if (useDllStorage())
860862
cast<llvm::GlobalVariable>(symbolAddr)
861863
->setDLLStorageClass(llvm::GlobalValue::DLLImportStorageClass);
862864

@@ -921,9 +923,9 @@ void IRGenModule::emitAutolinkInfo() {
921923
}),
922924
AutolinkEntries.end());
923925

924-
if (TargetInfo.OutputObjectFormat == llvm::Triple::COFF ||
925-
TargetInfo.OutputObjectFormat == llvm::Triple::MachO ||
926-
Triple.isPS4()) {
926+
if ((TargetInfo.OutputObjectFormat == llvm::Triple::COFF &&
927+
!Triple.isOSCygMing()) ||
928+
TargetInfo.OutputObjectFormat == llvm::Triple::MachO || Triple.isPS4()) {
927929
llvm::LLVMContext &ctx = Module.getContext();
928930

929931
if (!LinkerOptions) {
@@ -940,8 +942,9 @@ void IRGenModule::emitAutolinkInfo() {
940942
assert(FoundOldEntry && "Could not replace old linker options entry?");
941943
}
942944
} else {
943-
assert(TargetInfo.OutputObjectFormat == llvm::Triple::ELF &&
944-
"expected ELF output format");
945+
assert((TargetInfo.OutputObjectFormat == llvm::Triple::ELF ||
946+
Triple.isOSCygMing()) &&
947+
"expected ELF output format or COFF format for Cygwin/MinGW");
945948

946949
// Merge the entries into null-separated string.
947950
llvm::SmallString<64> EntriesString;
@@ -974,7 +977,7 @@ void IRGenModule::emitAutolinkInfo() {
974977
llvm::GlobalValue::CommonLinkage,
975978
llvm::Constant::getNullValue(Int1Ty),
976979
buf.str());
977-
if (Triple.isOSBinFormatCOFF())
980+
if (useDllStorage())
978981
symbol->setDLLStorageClass(llvm::GlobalValue::DLLExportStorageClass);
979982
}
980983
}

‎lib/IRGen/IRGenModule.h‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -520,6 +520,10 @@ class IRGenModule {
520520
void fatal_unimplemented(SourceLoc, StringRef Message);
521521
void error(SourceLoc loc, const Twine &message);
522522

523+
bool useDllStorage() {
524+
return Triple.isOSBinFormatCOFF() && !Triple.isOSCygMing();
525+
}
526+
523527
private:
524528
Size PtrSize;
525529
llvm::Type *FixedBufferTy; /// [N x i8], where N == 3 * sizeof(void*)

‎lib/IRGen/TypeLayoutVerifier.cpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ irgen::emitTypeLayoutVerifier(IRGenFunction &IGF,
4343
/*var arg*/ false);
4444
auto verifierFn = IGF.IGM.Module.getOrInsertFunction(
4545
"_swift_debug_verifyTypeLayoutAttribute", verifierFnTy);
46-
if (IGF.IGM.Triple.isOSBinFormatCOFF())
46+
if (IGF.IGM.Triple.isOSBinFormatCOFF() && !IGF.IGM.Triple.isOSCygMing())
4747
if (auto *F = dyn_cast<llvm::Function>(verifierFn))
4848
F->setDLLStorageClass(llvm::GlobalValue::DLLImportStorageClass);
4949

0 commit comments

Comments
 (0)