[PATCH] D80143: [ELF] Make --trace-symbol track preempted shared definitions

Peter Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 19 01:03:29 PDT 2020


psmith accepted this revision.
psmith added a comment.
This revision is now accepted and ready to land.

LGTM.



================
Comment at: lld/ELF/Symbols.cpp:716
 
 void Symbol::resolveShared(const SharedSymbol &other) {
   if (isCommon()) {
----------------
I know some prefer an unrelated refactoring change to be split out. Maybe worth committing this separately as an NFC white space change.




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80143





More information about the llvm-commits mailing list