Index: clang/lib/CodeGen/CodeGenModule.cpp =================================================================== --- clang/lib/CodeGen/CodeGenModule.cpp +++ clang/lib/CodeGen/CodeGenModule.cpp @@ -6013,10 +6013,13 @@ getModule(), Type, Constant, Linkage, InitialValue, Name.c_str(), /*InsertBefore=*/nullptr, llvm::GlobalVariable::NotThreadLocal, TargetAS); if (emitter) emitter->finalize(GV); - setGVProperties(GV, VD); - if (GV->getDLLStorageClass() == llvm::GlobalVariable::DLLExportStorageClass) - // The reference temporary should never be dllexport. - GV->setDLLStorageClass(llvm::GlobalVariable::DefaultStorageClass); + // Don't assign dllimport or dllexport to local linkage globals. + if (!llvm::GlobalValue::isLocalLinkage(Linkage)) { + setGVProperties(GV, VD); + if (GV->getDLLStorageClass() == llvm::GlobalVariable::DLLExportStorageClass) + // The reference temporary should never be dllexport. + GV->setDLLStorageClass(llvm::GlobalVariable::DefaultStorageClass); + } GV->setAlignment(Align.getAsAlign()); if (supportsCOMDAT() && GV->isWeakForLinker()) GV->setComdat(TheModule.getOrInsertComdat(GV->getName())); Index: llvm/docs/LangRef.rst =================================================================== --- llvm/docs/LangRef.rst +++ llvm/docs/LangRef.rst @@ -522,6 +522,9 @@ assembler and linker know it is externally referenced and must refrain from deleting the symbol. +A symbol with ``internal`` or ``private`` linkage cannot have a DLL storage +class. + .. _tls_model: Thread Local Storage Models Index: llvm/include/llvm/IR/GlobalValue.h =================================================================== --- llvm/include/llvm/IR/GlobalValue.h +++ llvm/include/llvm/IR/GlobalValue.h @@ -275,7 +275,11 @@ bool hasDLLExportStorageClass() const { return DllStorageClass == DLLExportStorageClass; } - void setDLLStorageClass(DLLStorageClassTypes C) { DllStorageClass = C; } + void setDLLStorageClass(DLLStorageClassTypes C) { + assert((!hasLocalLinkage() || C == DefaultStorageClass) && + "local linkage requires DefaultStorageClass"); + DllStorageClass = C; + } bool hasSection() const { return !getSection().empty(); } StringRef getSection() const; @@ -524,8 +528,10 @@ } void setLinkage(LinkageTypes LT) { - if (isLocalLinkage(LT)) + if (isLocalLinkage(LT)) { Visibility = DefaultVisibility; + DllStorageClass = DefaultStorageClass; + } Linkage = LT; if (isImplicitDSOLocal()) setDSOLocal(true); Index: llvm/lib/AsmParser/LLParser.cpp =================================================================== --- llvm/lib/AsmParser/LLParser.cpp +++ llvm/lib/AsmParser/LLParser.cpp @@ -929,6 +929,10 @@ return !GlobalValue::isLocalLinkage((GlobalValue::LinkageTypes)L) || (GlobalValue::VisibilityTypes)V == GlobalValue::DefaultVisibility; } +static bool isValidDLLStorageClassForLinkage(unsigned S, unsigned L) { + return !GlobalValue::isLocalLinkage((GlobalValue::LinkageTypes)L) || + (GlobalValue::DLLStorageClassTypes)S == GlobalValue::DefaultStorageClass; +} // If there was an explicit dso_local, update GV. In the absence of an explicit // dso_local we keep the default value. @@ -982,6 +986,10 @@ return error(NameLoc, "symbol with local linkage must have default visibility"); + if (!isValidDLLStorageClassForLinkage(DLLStorageClass, L)) + return error(NameLoc, + "symbol with local linkage cannot have a DLL storage class"); + Type *Ty; LocTy ExplicitTypeLoc = Lex.getLoc(); if (parseType(Ty) || @@ -1169,6 +1177,10 @@ return error(NameLoc, "symbol with local linkage must have default visibility"); + if (!isValidDLLStorageClassForLinkage(DLLStorageClass, Linkage)) + return error(NameLoc, + "symbol with local linkage cannot have a DLL storage class"); + unsigned AddrSpace; bool IsConstant, IsExternallyInitialized; LocTy IsExternallyInitializedLoc; @@ -5603,6 +5615,10 @@ return error(LinkageLoc, "symbol with local linkage must have default visibility"); + if (!isValidDLLStorageClassForLinkage(DLLStorageClass, Linkage)) + return error(LinkageLoc, + "symbol with local linkage cannot have a DLL storage class"); + if (!FunctionType::isValidReturnType(RetType)) return error(RetTypeLoc, "invalid function return type"); Index: llvm/lib/Bitcode/Reader/BitcodeReader.cpp =================================================================== --- llvm/lib/Bitcode/Reader/BitcodeReader.cpp +++ llvm/lib/Bitcode/Reader/BitcodeReader.cpp @@ -1298,6 +1298,9 @@ } static void upgradeDLLImportExportLinkage(GlobalValue *GV, unsigned Val) { + // A GlobalValue with local linkage cannot have a DLL storage class. + if (GV->hasLocalLinkage()) + return; switch (Val) { case 5: GV->setDLLStorageClass(GlobalValue::DLLImportStorageClass); break; case 6: GV->setDLLStorageClass(GlobalValue::DLLExportStorageClass); break; @@ -3764,10 +3767,14 @@ NewGV->setVisibility(Visibility); NewGV->setUnnamedAddr(UnnamedAddr); - if (Record.size() > 10) - NewGV->setDLLStorageClass(getDecodedDLLStorageClass(Record[10])); - else + if (Record.size() > 10) { + // A GlobalValue with local linkage cannot have a DLL storage class. + if (!NewGV->hasLocalLinkage()) { + NewGV->setDLLStorageClass(getDecodedDLLStorageClass(Record[10])); + } + } else { upgradeDLLImportExportLinkage(NewGV, RawLinkage); + } ValueList.push_back(NewGV, getVirtualTypeID(NewGV->getType(), TyID)); @@ -3928,10 +3935,14 @@ if (Record.size() > 10) OperandInfo.Prologue = Record[10]; - if (Record.size() > 11) - Func->setDLLStorageClass(getDecodedDLLStorageClass(Record[11])); - else + if (Record.size() > 11) { + // A GlobalValue with local linkage cannot have a DLL storage class. + if (!Func->hasLocalLinkage()) { + Func->setDLLStorageClass(getDecodedDLLStorageClass(Record[11])); + } + } else { upgradeDLLImportExportLinkage(Func, RawLinkage); + } if (Record.size() > 12) { if (unsigned ComdatID = Record[12]) { @@ -4034,8 +4045,12 @@ } if (BitCode == bitc::MODULE_CODE_ALIAS || BitCode == bitc::MODULE_CODE_ALIAS_OLD) { - if (OpNum != Record.size()) - NewGA->setDLLStorageClass(getDecodedDLLStorageClass(Record[OpNum++])); + if (OpNum != Record.size()) { + auto S = Record[OpNum++]; + // A GlobalValue with local linkage cannot have a DLL storage class. + if (!NewGA->hasLocalLinkage()) + NewGA->setDLLStorageClass(getDecodedDLLStorageClass(S)); + } else upgradeDLLImportExportLinkage(NewGA, Linkage); if (OpNum != Record.size()) Index: llvm/test/Assembler/dll-storage-class-local-linkage.ll =================================================================== --- /dev/null +++ llvm/test/Assembler/dll-storage-class-local-linkage.ll @@ -0,0 +1,88 @@ +;; Check that an error is emitted for local linkage symbols with DLL storage class. + +; RUN: rm -rf %t && split-file %s %t + +; RUN: not llvm-as %t/internal_function_dllexport.ll -o /dev/null 2>&1 | FileCheck %s +; RUN: not llvm-as %t/internal_variable_dllexport.ll -o /dev/null 2>&1 | FileCheck %s +; RUN: not llvm-as %t/internal_alias_dllexport.ll -o /dev/null 2>&1 | FileCheck %s + +; RUN: not llvm-as %t/private_function_dllexport.ll -o /dev/null 2>&1 | FileCheck %s +; RUN: not llvm-as %t/private_variable_dllexport.ll -o /dev/null 2>&1 | FileCheck %s +; RUN: not llvm-as %t/private_alias_dllexport.ll -o /dev/null 2>&1 | FileCheck %s + +; RUN: not llvm-as %t/internal_function_dllimport.ll -o /dev/null 2>&1 | FileCheck %s +; RUN: not llvm-as %t/internal_variable_dllimport.ll -o /dev/null 2>&1 | FileCheck %s +; RUN: not llvm-as %t/internal_alias_dllimport.ll -o /dev/null 2>&1 | FileCheck %s + +; RUN: not llvm-as %t/private_function_dllimport.ll -o /dev/null 2>&1 | FileCheck %s +; RUN: not llvm-as %t/private_variable_dllimport.ll -o /dev/null 2>&1 | FileCheck %s +; RUN: not llvm-as %t/private_alias_dllimport.ll -o /dev/null 2>&1 | FileCheck %s + + +; CHECK: symbol with local linkage cannot have a DLL storage class + + +;--- internal_function_dllexport.ll + +define internal dllexport void @function() { +entry: + ret void +} + +;--- internal_variable_dllexport.ll + +@var = internal dllexport global i32 0 + +;--- internal_alias_dllexport.ll + +@global = global i32 0 +@alias = internal dllexport alias i32, i32* @global + +;--- private_function_dllexport.ll + +define private dllexport void @function() { +entry: + ret void +} + +;--- private_variable_dllexport.ll + +@var = private dllexport global i32 0 + +;--- private_alias_dllexport.ll + +@global = global i32 0 +@alias = private dllexport alias i32, i32* @global + + +;--- internal_function_dllimport.ll + +define internal dllimport void @function() { +entry: + ret void +} + +;--- internal_variable_dllimport.ll + +@var = internal dllimport global i32 0 + +;--- internal_alias_dllimport.ll + +@global = global i32 0 +@alias = internal dllimport alias i32, i32* @global + +;--- private_function_dllimport.ll + +define private dllimport void @function() { +entry: + ret void +} + +;--- private_variable_dllimport.ll + +@var = private dllimport global i32 0 + +;--- private_alias_dllimport.ll + +@global = global i32 0 +@alias = private dllimport alias i32, i32* @global Index: llvm/test/Transforms/GlobalOpt/alias-used-section.ll =================================================================== --- llvm/test/Transforms/GlobalOpt/alias-used-section.ll +++ llvm/test/Transforms/GlobalOpt/alias-used-section.ll @@ -1,8 +1,8 @@ ; RUN: opt -S -passes=globalopt < %s | FileCheck %s @_Z17in_custom_section = internal global i8 42, section "CUSTOM" -@in_custom_section = internal dllexport alias i8, i8* @_Z17in_custom_section +@in_custom_section = internal alias i8, i8* @_Z17in_custom_section -; CHECK: @in_custom_section = internal dllexport global i8 42, section "CUSTOM" +; CHECK: @in_custom_section = internal global i8 42, section "CUSTOM" @llvm.used = appending global [1 x i8*] [i8* @in_custom_section], section "llvm.metadata"