Skip to content

Commit cc4254c

Browse files
authored
Merge pull request #11011 from artemcm/Next_FixGVLinkageForClangTarget
[`next` 🍒][clang] Fix missing target info specifier on Global Variable linkage computation
2 parents 926bbed + 69d0b90 commit cc4254c

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

clang/lib/CodeGen/CodeGenModule.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2987,7 +2987,7 @@ void CodeGenModule::SetInternalFunctionAttributes(GlobalDecl GD,
29872987
}
29882988

29892989
static void setLinkageForGV(llvm::GlobalValue *GV, const NamedDecl *ND,
2990-
VersionTuple EnclosingVersion = VersionTuple()) {
2990+
VersionTuple EnclosingVersion) {
29912991
// Set linkage and visibility in case we never see a definition.
29922992
LinkageInfo LV = ND->getLinkageAndVisibility();
29932993
// Don't set internal linkage on declarations.
@@ -5282,7 +5282,7 @@ CodeGenModule::GetOrCreateLLVMGlobal(StringRef MangledName, llvm::Type *Ty,
52825282

52835283
GV->setAlignment(getContext().getDeclAlign(D).getAsAlign());
52845284

5285-
setLinkageForGV(GV, D);
5285+
setLinkageForGV(GV, D, Target.getPlatformMinVersion());
52865286

52875287
if (D->getTLSKind()) {
52885288
if (D->getTLSKind() == VarDecl::TLS_Dynamic)

0 commit comments

Comments
 (0)