[lld] r231445 - We want to add the entry point to the root set unconditionally, but these
Davide Italiano
davide at freebsd.org
Thu Mar 5 18:05:03 PST 2015
Author: davide
Date: Thu Mar 5 20:05:03 2015
New Revision: 231445
URL: http://llvm.org/viewvc/llvm-project?rev=231445&view=rev
Log:
We want to add the entry point to the root set unconditionally, but these
asserts don't allow us to do it. Remove them, they're not really needed
anyway.
Modified:
lld/trunk/include/lld/Core/LinkingContext.h
Modified: lld/trunk/include/lld/Core/LinkingContext.h
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/include/lld/Core/LinkingContext.h?rev=231445&r1=231444&r2=231445&view=diff
==============================================================================
--- lld/trunk/include/lld/Core/LinkingContext.h (original)
+++ lld/trunk/include/lld/Core/LinkingContext.h Thu Mar 5 20:05:03 2015
@@ -62,24 +62,19 @@ public:
/// should be marked live (along with all Atoms they reference). Usually
/// this method returns false for main executables, but true for dynamic
/// shared libraries.
- bool globalsAreDeadStripRoots() const {
- assert(_deadStrip && "only applicable when deadstripping enabled");
- return _globalsAreDeadStripRoots;
- }
+ bool globalsAreDeadStripRoots() const { return _globalsAreDeadStripRoots; };
/// Only used if deadStrip() returns true. This method returns the names
/// of DefinedAtoms that should be marked live (along with all Atoms they
/// reference). Only Atoms with scope scopeLinkageUnit or scopeGlobal can
/// be kept live using this method.
const std::vector<StringRef> &deadStripRoots() const {
- assert(_deadStrip && "only applicable when deadstripping enabled");
return _deadStripRoots;
}
/// Add the given symbol name to the dead strip root set. Only used if
/// deadStrip() returns true.
void addDeadStripRoot(StringRef symbolName) {
- assert(_deadStrip && "only applicable when deadstripping enabled");
assert(!symbolName.empty() && "Empty symbol cannot be a dead strip root");
_deadStripRoots.push_back(symbolName);
}
More information about the llvm-commits
mailing list