[PATCH] [lld][ELF] Make AtomLayout more accessible.
    Michael Spencer 
    bigcheesegs at gmail.com
       
    Tue Jan 29 00:57:48 PST 2013
    
    
  
================
Comment at: lib/ReaderWriter/ELF/DefaultELFLayout.h:379-380
@@ -397,1 +378,4 @@
+    // -noinhibit-exec.
+    if (definedAtom->contentType() == DefinedAtom::typeUnknown)
+      return make_error_code(llvm::errc::invalid_argument);
     const StringRef sectionName = getSectionName(
----------------
Shankar Kalpathi Easwaran wrote:
> Is there a need to check this ?
Yes, we can't return an AtomLayout for an undefined symbol.
http://llvm-reviews.chandlerc.com/D340
    
    
More information about the llvm-commits
mailing list