[PATCH] D51892: Revert "Also demote lazy symbols."

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 11 13:41:08 PDT 2018


ruiu added a comment.

We generally don't check in a binary file unless there's no way to generate a file from text. Looks like we already have a test that creates a .gnu.linkonce section, see test/ELF/comdat-linkonce.s. Could you create a test based on it?



================
Comment at: test/ELF/i386-linkonce.s:3
+// RUN: llvm-mc -filetype=obj -triple=i386-linux-gnu %s -o %t.o
+// RUN: not ld.lld %t.o %p/Inputs/i386-glibc-strchr.a -o %t 2>&1 | FileCheck %s
+
----------------
Please write a test that is expected to succeed to make it more reliable.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D51892





More information about the llvm-commits mailing list