[PATCH] D63752: Fix an issue that common symbols are not internalized under some condition.

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 24 23:58:34 PDT 2019


This revision was automatically updated to reflect the committed changes.
Closed by commit rL364273: Fix an issue that common symbols are not internalized under some condition. (authored by ruiu, committed by ).

Changed prior to commit:
  https://reviews.llvm.org/D63752?vs=206367&id=206369#toc

Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D63752/new/

https://reviews.llvm.org/D63752

Files:
  lld/trunk/ELF/Symbols.cpp
  lld/trunk/test/ELF/lto/common4.ll


Index: lld/trunk/ELF/Symbols.cpp
===================================================================
--- lld/trunk/ELF/Symbols.cpp
+++ lld/trunk/ELF/Symbols.cpp
@@ -275,13 +275,13 @@
     return false;
   if (computeBinding() == STB_LOCAL)
     return false;
+
   // If a PIE binary was not linked against any shared libraries, then we can
   // safely drop weak undef symbols from .dynsym.
   if (isUndefWeak() && Config->Pie && SharedFiles.empty())
     return false;
-  if (!isDefined())
-    return true;
-  return ExportDynamic;
+
+  return isUndefined() || isShared() || ExportDynamic;
 }
 
 // Print out a log message for --trace-symbol.
Index: lld/trunk/test/ELF/lto/common4.ll
===================================================================
--- lld/trunk/test/ELF/lto/common4.ll
+++ lld/trunk/test/ELF/lto/common4.ll
@@ -0,0 +1,21 @@
+; REQUIRES: x86
+
+;; Make sure that common symbols are properly internalized.
+;; In this file, @a does not interpose any symbol in a DSO,
+;; so LTO should be able to internelize it.
+
+; RUN: llvm-mc -filetype=obj -triple=x86_64-unknown-linux /dev/null -o %t.so.o
+; RUN: ld.lld -shared -o %t.so %t.so.o
+
+; RUN: llvm-as %s -o %t.o
+; RUN: ld.lld -o %t.exe -save-temps %t.o %t.so
+; RUN: llvm-dis < %t.exe.0.2.internalize.bc | FileCheck %s
+
+; RUN: ld.lld -pie -o %t.exe -save-temps %t.o
+; RUN: llvm-dis < %t.exe.0.2.internalize.bc | FileCheck %s
+
+target datalayout = "e-m:e-i64:64-f80:128-n8:16:32:64-S128"
+target triple = "x86_64-unknown-linux-gnu"
+
+ at a = common dso_local local_unnamed_addr global i32 0, align 4
+; CHECK-DAG: @a = internal global i32 0, align 4


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D63752.206369.patch
Type: text/x-patch
Size: 1624 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190625/1d277331/attachment.bin>


More information about the llvm-commits mailing list