summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFangrui Song <maskray@google.com>2018-04-20 22:50:15 +0000
committerFangrui Song <maskray@google.com>2018-04-20 22:50:15 +0000
commitc8ac0a6f979c36ad7f7c3e1b9b2e8260f483aa15 (patch)
tree66d2d6c97d5ec408c0f0259ac29afa01f8efb512
parent8679a7ecea9b630d7757075e18a4727a5d6a5fa5 (diff)
downloadbcm5719-llvm-c8ac0a6f979c36ad7f7c3e1b9b2e8260f483aa15.tar.gz
bcm5719-llvm-c8ac0a6f979c36ad7f7c3e1b9b2e8260f483aa15.zip
[ELF] Swap argument names: use Old to refer to original symbol and New for incoming one
Reviewers: ruiu, espindola Subscribers: emaste, arichardson, llvm-commits Differential Revision: https://reviews.llvm.org/D45905 llvm-svn: 330491
-rw-r--r--lld/ELF/SymbolTable.cpp8
1 files changed, 4 insertions, 4 deletions
diff --git a/lld/ELF/SymbolTable.cpp b/lld/ELF/SymbolTable.cpp
index 041aeb66d5c..49aa9086f56 100644
--- a/lld/ELF/SymbolTable.cpp
+++ b/lld/ELF/SymbolTable.cpp
@@ -339,9 +339,9 @@ static uint8_t getVisibility(uint8_t StOther) { return StOther & 3; }
// files) form group 2. E forms group 3. I think that you can see how this
// group assignment rule simulates the traditional linker's semantics.
static void checkBackrefs(StringRef Name, InputFile *Old, InputFile *New) {
- if (Config->WarnBackrefs && Old && New->GroupId < Old->GroupId)
- warn("backward reference detected: " + Name + " in " + toString(Old) +
- " refers to " + toString(New));
+ if (Config->WarnBackrefs && New && Old->GroupId < New->GroupId)
+ warn("backward reference detected: " + Name + " in " + toString(New) +
+ " refers to " + toString(Old));
}
template <class ELFT>
@@ -376,7 +376,7 @@ Symbol *SymbolTable::addUndefined(StringRef Name, uint8_t Binding,
return S;
}
- checkBackrefs(Name, File, S->File);
+ checkBackrefs(Name, S->File, File);
fetchLazy<ELFT>(S);
}
return S;
OpenPOWER on IntegriCloud