[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