This seems to have caused two copies of getElfArchType to appear in the tree. One in ELF.h and one in lib/Object/ELFObjectWriter.cpp. The code in that file is now finding the copy in ELF.h and leaving the copy in the anonymous namespace in ELFObjectWriter unused leading to a warning.<br>
<br><div class="gmail_quote">On Thu, Apr 12, 2012 at 1:13 PM, Preston Gurd <span dir="ltr"><<a href="mailto:preston.gurd@intel.com">preston.gurd@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Author: pgurd<br>
Date: Thu Apr 12 15:13:57 2012<br>
New Revision: 154610<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=154610&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=154610&view=rev</a><br>
Log:<br>
This patch improves the MCJIT runtime dynamic loader by adding new handling<br>
of zero-initialized sections, virtual sections and common symbols<br>
and preventing the loading of sections which are not required for<br>
execution such as debug information.<br>
<br>
Patch by Andy Kaylor!<br>
<br>
<br>
Modified:<br>
llvm/trunk/include/llvm/Object/COFF.h<br>
llvm/trunk/include/llvm/Object/ELF.h<br>
llvm/trunk/include/llvm/Object/MachO.h<br>
llvm/trunk/include/llvm/Object/ObjectFile.h<br>
llvm/trunk/lib/ExecutionEngine/MCJIT/MCJIT.cpp<br>
llvm/trunk/lib/ExecutionEngine/RuntimeDyld/RuntimeDyld.cpp<br>
llvm/trunk/lib/ExecutionEngine/RuntimeDyld/RuntimeDyldELF.cpp<br>
llvm/trunk/lib/ExecutionEngine/RuntimeDyld/RuntimeDyldImpl.h<br>
llvm/trunk/lib/Object/COFFObjectFile.cpp<br>
llvm/trunk/lib/Object/MachOObjectFile.cpp<br>
llvm/trunk/test/ExecutionEngine/2002-12-16-ArgTest.ll<br>
llvm/trunk/test/ExecutionEngine/2003-01-04-ArgumentBug.ll<br>
llvm/trunk/test/ExecutionEngine/2003-01-04-LoopTest.ll<br>
llvm/trunk/test/ExecutionEngine/2003-05-06-LivenessClobber.ll<br>
llvm/trunk/test/ExecutionEngine/2003-05-07-ArgumentTest.ll<br>
llvm/trunk/test/ExecutionEngine/2003-08-21-EnvironmentTest.ll<br>
llvm/trunk/test/ExecutionEngine/2005-12-02-TailCallBug.ll<br>
llvm/trunk/test/ExecutionEngine/hello.ll<br>
llvm/trunk/test/ExecutionEngine/hello2.ll<br>
llvm/trunk/test/ExecutionEngine/stubs.ll<br>
llvm/trunk/test/ExecutionEngine/test-call-no-external-funcs.ll<br>
llvm/trunk/test/ExecutionEngine/test-call.ll<br>
llvm/trunk/test/ExecutionEngine/test-common-symbols.ll<br>
llvm/trunk/test/ExecutionEngine/test-fp-no-external-funcs.ll<br>
llvm/trunk/test/ExecutionEngine/test-fp.ll<br>
llvm/trunk/test/ExecutionEngine/test-global-init-nonzero.ll<br>
llvm/trunk/test/ExecutionEngine/test-global.ll<br>
llvm/trunk/test/ExecutionEngine/test-loadstore.ll<br>
<br>
Modified: llvm/trunk/include/llvm/Object/COFF.h<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Object/COFF.h?rev=154610&r1=154609&r2=154610&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Object/COFF.h?rev=154610&r1=154609&r2=154610&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/include/llvm/Object/COFF.h (original)<br>
+++ llvm/trunk/include/llvm/Object/COFF.h Thu Apr 12 15:13:57 2012<br>
@@ -126,6 +126,10 @@<br>
virtual error_code isSectionText(DataRefImpl Sec, bool &Res) const;<br>
virtual error_code isSectionData(DataRefImpl Sec, bool &Res) const;<br>
virtual error_code isSectionBSS(DataRefImpl Sec, bool &Res) const;<br>
+ virtual error_code isSectionVirtual(DataRefImpl Sec, bool &Res) const;<br>
+ virtual error_code isSectionZeroInit(DataRefImpl Sec, bool &Res) const;<br>
+ virtual error_code isSectionRequiredForExecution(DataRefImpl Sec,<br>
+ bool &Res) const;<br>
virtual error_code sectionContainsSymbol(DataRefImpl Sec, DataRefImpl Symb,<br>
bool &Result) const;<br>
virtual relocation_iterator getSectionRelBegin(DataRefImpl Sec) const;<br>
<br>
Modified: llvm/trunk/include/llvm/Object/ELF.h<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Object/ELF.h?rev=154610&r1=154609&r2=154610&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Object/ELF.h?rev=154610&r1=154609&r2=154610&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/include/llvm/Object/ELF.h (original)<br>
+++ llvm/trunk/include/llvm/Object/ELF.h Thu Apr 12 15:13:57 2012<br>
@@ -33,6 +33,15 @@<br>
namespace llvm {<br>
namespace object {<br>
<br>
+// Subclasses of ELFObjectFile may need this for template instantiation<br>
+inline std::pair<unsigned char, unsigned char><br>
+getElfArchType(MemoryBuffer *Object) {<br>
+ if (Object->getBufferSize() < ELF::EI_NIDENT)<br>
+ return std::make_pair((uint8_t)ELF::ELFCLASSNONE,(uint8_t)ELF::ELFDATANONE);<br>
+ return std::make_pair( (uint8_t)Object->getBufferStart()[ELF::EI_CLASS]<br>
+ , (uint8_t)Object->getBufferStart()[ELF::EI_DATA]);<br>
+}<br>
+<br>
// Templates to choose Elf_Addr and Elf_Off depending on is64Bits.<br>
template<support::endianness target_endianness><br>
struct ELFDataTypeTypedefHelperCommon {<br>
@@ -540,6 +549,10 @@<br>
virtual error_code isSectionText(DataRefImpl Sec, bool &Res) const;<br>
virtual error_code isSectionData(DataRefImpl Sec, bool &Res) const;<br>
virtual error_code isSectionBSS(DataRefImpl Sec, bool &Res) const;<br>
+ virtual error_code isSectionRequiredForExecution(DataRefImpl Sec,<br>
+ bool &Res) const;<br>
+ virtual error_code isSectionVirtual(DataRefImpl Sec, bool &Res) const;<br>
+ virtual error_code isSectionZeroInit(DataRefImpl Sec, bool &Res) const;<br>
virtual error_code sectionContainsSymbol(DataRefImpl Sec, DataRefImpl Symb,<br>
bool &Result) const;<br>
virtual relocation_iterator getSectionRelBegin(DataRefImpl Sec) const;<br>
@@ -1094,6 +1107,43 @@<br>
<br>
template<support::endianness target_endianness, bool is64Bits><br>
error_code ELFObjectFile<target_endianness, is64Bits><br>
+ ::isSectionRequiredForExecution(DataRefImpl Sec,<br>
+ bool &Result) const {<br>
+ const Elf_Shdr *sec = reinterpret_cast<const Elf_Shdr *>(Sec.p);<br>
+ if (sec->sh_flags & ELF::SHF_ALLOC)<br>
+ Result = true;<br>
+ else<br>
+ Result = false;<br>
+ return object_error::success;<br>
+}<br>
+<br>
+template<support::endianness target_endianness, bool is64Bits><br>
+error_code ELFObjectFile<target_endianness, is64Bits><br>
+ ::isSectionVirtual(DataRefImpl Sec,<br>
+ bool &Result) const {<br>
+ const Elf_Shdr *sec = reinterpret_cast<const Elf_Shdr *>(Sec.p);<br>
+ if (sec->sh_type == ELF::SHT_NOBITS)<br>
+ Result = true;<br>
+ else<br>
+ Result = false;<br>
+ return object_error::success;<br>
+}<br>
+<br>
+template<support::endianness target_endianness, bool is64Bits><br>
+error_code ELFObjectFile<target_endianness, is64Bits>::isSectionZeroInit(DataRefImpl Sec,<br>
+ bool &Result) const {<br>
+ const Elf_Shdr *sec = reinterpret_cast<const Elf_Shdr *>(Sec.p);<br>
+ // For ELF, all zero-init sections are virtual (that is, they occupy no space<br>
+ // in the object image) and vice versa.<br>
+ if (sec->sh_flags & ELF::SHT_NOBITS)<br>
+ Result = true;<br>
+ else<br>
+ Result = false;<br>
+ return object_error::success;<br>
+}<br>
+<br>
+template<support::endianness target_endianness, bool is64Bits><br>
+error_code ELFObjectFile<target_endianness, is64Bits><br>
::sectionContainsSymbol(DataRefImpl Sec,<br>
DataRefImpl Symb,<br>
bool &Result) const {<br>
<br>
Modified: llvm/trunk/include/llvm/Object/MachO.h<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Object/MachO.h?rev=154610&r1=154609&r2=154610&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Object/MachO.h?rev=154610&r1=154609&r2=154610&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/include/llvm/Object/MachO.h (original)<br>
+++ llvm/trunk/include/llvm/Object/MachO.h Thu Apr 12 15:13:57 2012<br>
@@ -72,6 +72,10 @@<br>
virtual error_code isSectionText(DataRefImpl Sec, bool &Res) const;<br>
virtual error_code isSectionData(DataRefImpl Sec, bool &Res) const;<br>
virtual error_code isSectionBSS(DataRefImpl Sec, bool &Res) const;<br>
+ virtual error_code isSectionRequiredForExecution(DataRefImpl Sec,<br>
+ bool &Res) const;<br>
+ virtual error_code isSectionVirtual(DataRefImpl Sec, bool &Res) const;<br>
+ virtual error_code isSectionZeroInit(DataRefImpl Sec, bool &Res) const;<br>
virtual error_code sectionContainsSymbol(DataRefImpl DRI, DataRefImpl S,<br>
bool &Result) const;<br>
virtual relocation_iterator getSectionRelBegin(DataRefImpl Sec) const;<br>
<br>
Modified: llvm/trunk/include/llvm/Object/ObjectFile.h<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Object/ObjectFile.h?rev=154610&r1=154609&r2=154610&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Object/ObjectFile.h?rev=154610&r1=154609&r2=154610&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/include/llvm/Object/ObjectFile.h (original)<br>
+++ llvm/trunk/include/llvm/Object/ObjectFile.h Thu Apr 12 15:13:57 2012<br>
@@ -158,11 +158,16 @@<br>
error_code isText(bool &Result) const;<br>
error_code isData(bool &Result) const;<br>
error_code isBSS(bool &Result) const;<br>
+ error_code isRequiredForExecution(bool &Result) const;<br>
+ error_code isVirtual(bool &Result) const;<br>
+ error_code isZeroInit(bool &Result) const;<br>
<br>
error_code containsSymbol(SymbolRef S, bool &Result) const;<br>
<br>
relocation_iterator begin_relocations() const;<br>
relocation_iterator end_relocations() const;<br>
+<br>
+ DataRefImpl getRawDataRefImpl() const;<br>
};<br>
typedef content_iterator<SectionRef> section_iterator;<br>
<br>
@@ -217,6 +222,9 @@<br>
/// Get symbol flags (bitwise OR of SymbolRef::Flags)<br>
error_code getFlags(uint32_t &Result) const;<br>
<br>
+ /// @brief Return true for common symbols such as uninitialized globals<br>
+ error_code isCommon(bool &Result) const;<br>
+<br>
/// @brief Get section this symbol is defined in reference to. Result is<br>
/// end_sections() if it is undefined or is an absolute symbol.<br>
error_code getSection(section_iterator &Result) const;<br>
@@ -299,6 +307,11 @@<br>
virtual error_code isSectionText(DataRefImpl Sec, bool &Res) const = 0;<br>
virtual error_code isSectionData(DataRefImpl Sec, bool &Res) const = 0;<br>
virtual error_code isSectionBSS(DataRefImpl Sec, bool &Res) const = 0;<br>
+ virtual error_code isSectionRequiredForExecution(DataRefImpl Sec,<br>
+ bool &Res) const = 0;<br>
+ // A section is 'virtual' if its contents aren't present in the object image.<br>
+ virtual error_code isSectionVirtual(DataRefImpl Sec, bool &Res) const = 0;<br>
+ virtual error_code isSectionZeroInit(DataRefImpl Sec, bool &Res) const = 0;<br>
virtual error_code sectionContainsSymbol(DataRefImpl Sec, DataRefImpl Symb,<br>
bool &Result) const = 0;<br>
virtual relocation_iterator getSectionRelBegin(DataRefImpl Sec) const = 0;<br>
@@ -481,6 +494,18 @@<br>
return OwningObject->isSectionBSS(SectionPimpl, Result);<br>
}<br>
<br>
+inline error_code SectionRef::isRequiredForExecution(bool &Result) const {<br>
+ return OwningObject->isSectionRequiredForExecution(SectionPimpl, Result);<br>
+}<br>
+<br>
+inline error_code SectionRef::isVirtual(bool &Result) const {<br>
+ return OwningObject->isSectionVirtual(SectionPimpl, Result);<br>
+}<br>
+<br>
+inline error_code SectionRef::isZeroInit(bool &Result) const {<br>
+ return OwningObject->isSectionZeroInit(SectionPimpl, Result);<br>
+}<br>
+<br>
inline error_code SectionRef::containsSymbol(SymbolRef S, bool &Result) const {<br>
return OwningObject->sectionContainsSymbol(SectionPimpl, S.SymbolPimpl,<br>
Result);<br>
@@ -494,6 +519,9 @@<br>
return OwningObject->getSectionRelEnd(SectionPimpl);<br>
}<br>
<br>
+inline DataRefImpl SectionRef::getRawDataRefImpl() const {<br>
+ return SectionPimpl;<br>
+}<br>
<br>
/// RelocationRef<br>
inline RelocationRef::RelocationRef(DataRefImpl RelocationP,<br>
<br>
Modified: llvm/trunk/lib/ExecutionEngine/MCJIT/MCJIT.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/ExecutionEngine/MCJIT/MCJIT.cpp?rev=154610&r1=154609&r2=154610&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/ExecutionEngine/MCJIT/MCJIT.cpp?rev=154610&r1=154609&r2=154610&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/ExecutionEngine/MCJIT/MCJIT.cpp (original)<br>
+++ llvm/trunk/lib/ExecutionEngine/MCJIT/MCJIT.cpp Thu Apr 12 15:13:57 2012<br>
@@ -74,9 +74,9 @@<br>
OS.flush();<br>
<br>
// Load the object into the dynamic linker.<br>
- // FIXME: It would be nice to avoid making yet another copy.<br>
- MemoryBuffer *MB = MemoryBuffer::getMemBufferCopy(StringRef(Buffer.data(),<br>
- Buffer.size()));<br>
+ MemoryBuffer *MB = MemoryBuffer::getMemBuffer(StringRef(Buffer.data(),<br>
+ Buffer.size()),<br>
+ "", false);<br>
if (Dyld.loadObject(MB))<br>
report_fatal_error(Dyld.getErrorString());<br>
// Resolve any relocations.<br>
<br>
Modified: llvm/trunk/lib/ExecutionEngine/RuntimeDyld/RuntimeDyld.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/ExecutionEngine/RuntimeDyld/RuntimeDyld.cpp?rev=154610&r1=154609&r2=154610&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/ExecutionEngine/RuntimeDyld/RuntimeDyld.cpp?rev=154610&r1=154609&r2=154610&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/ExecutionEngine/RuntimeDyld/RuntimeDyld.cpp (original)<br>
+++ llvm/trunk/lib/ExecutionEngine/RuntimeDyld/RuntimeDyld.cpp Thu Apr 12 15:13:57 2012<br>
@@ -36,10 +36,9 @@<br>
}<br>
} // end anonymous namespace<br>
<br>
-<br>
// Resolve the relocations for all symbols we currently know about.<br>
void RuntimeDyldImpl::resolveRelocations() {<br>
- // First, resolve relocations assotiated with external symbols.<br>
+ // First, resolve relocations associated with external symbols.<br>
resolveSymbols();<br>
<br>
// Just iterate over the sections we have and resolve all the relocations<br>
@@ -63,14 +62,18 @@<br>
bool RuntimeDyldImpl::loadObject(const MemoryBuffer *InputBuffer) {<br>
// FIXME: ObjectFile don't modify MemoryBuffer.<br>
// It should use const MemoryBuffer as parameter.<br>
- ObjectFile *obj<br>
- = ObjectFile::createObjectFile(const_cast<MemoryBuffer*>(InputBuffer));<br>
+ OwningPtr<ObjectFile> obj(ObjectFile::createObjectFile(<br>
+ const_cast<MemoryBuffer*>(InputBuffer)));<br>
+ if (!obj)<br>
+ report_fatal_error("Unable to create object image from memory buffer!");<br>
<br>
Arch = (Triple::ArchType)obj->getArch();<br>
<br>
LocalSymbolMap LocalSymbols; // Functions and data symbols from the<br>
// object file.<br>
ObjSectionToIDMap LocalSections; // Used sections from the object file<br>
+ CommonSymbolMap CommonSymbols; // Common symbols requiring allocation<br>
+ uint64_t CommonSize = 0;<br>
<br>
error_code err;<br>
// Parse symbols<br>
@@ -83,36 +86,50 @@<br>
Check(i->getType(SymType));<br>
Check(i->getName(Name));<br>
<br>
- if (SymType == object::SymbolRef::ST_Function ||<br>
- SymType == object::SymbolRef::ST_Data) {<br>
- uint64_t FileOffset;<br>
- uint32_t flags;<br>
- StringRef sData;<br>
- section_iterator si = obj->end_sections();<br>
- Check(i->getFileOffset(FileOffset));<br>
- Check(i->getFlags(flags));<br>
- Check(i->getSection(si));<br>
- if (si == obj->end_sections()) continue;<br>
- Check(si->getContents(sData));<br>
- const uint8_t* SymPtr = (const uint8_t*)InputBuffer->getBufferStart() +<br>
- (uintptr_t)FileOffset;<br>
- uintptr_t SectOffset = (uintptr_t)(SymPtr - (const uint8_t*)sData.begin());<br>
- unsigned SectionID<br>
- = findOrEmitSection(*si,<br>
- SymType == object::SymbolRef::ST_Function,<br>
- LocalSections);<br>
- bool isGlobal = flags & SymbolRef::SF_Global;<br>
- LocalSymbols[Name.data()] = SymbolLoc(SectionID, SectOffset);<br>
- DEBUG(dbgs() << "\tFileOffset: " << format("%p", (uintptr_t)FileOffset)<br>
- << " flags: " << flags<br>
- << " SID: " << SectionID<br>
- << " Offset: " << format("%p", SectOffset));<br>
- if (isGlobal)<br>
- SymbolTable[Name] = SymbolLoc(SectionID, SectOffset);<br>
+ uint32_t flags;<br>
+ Check(i->getFlags(flags));<br>
+<br>
+ bool isCommon = flags & SymbolRef::SF_Common;<br>
+ if (isCommon) {<br>
+ // Add the common symbols to a list. We'll allocate them all below.<br>
+ uint64_t Size = 0;<br>
+ Check(i->getSize(Size));<br>
+ CommonSize += Size;<br>
+ CommonSymbols[*i] = Size;<br>
+ } else {<br>
+ if (SymType == object::SymbolRef::ST_Function ||<br>
+ SymType == object::SymbolRef::ST_Data) {<br>
+ uint64_t FileOffset;<br>
+ StringRef sData;<br>
+ section_iterator si = obj->end_sections();<br>
+ Check(i->getFileOffset(FileOffset));<br>
+ Check(i->getSection(si));<br>
+ if (si == obj->end_sections()) continue;<br>
+ Check(si->getContents(sData));<br>
+ const uint8_t* SymPtr = (const uint8_t*)InputBuffer->getBufferStart() +<br>
+ (uintptr_t)FileOffset;<br>
+ uintptr_t SectOffset = (uintptr_t)(SymPtr - (const uint8_t*)sData.begin());<br>
+ unsigned SectionID =<br>
+ findOrEmitSection(*si,<br>
+ SymType == object::SymbolRef::ST_Function,<br>
+ LocalSections);<br>
+ bool isGlobal = flags & SymbolRef::SF_Global;<br>
+ LocalSymbols[Name.data()] = SymbolLoc(SectionID, SectOffset);<br>
+ DEBUG(dbgs() << "\tFileOffset: " << format("%p", (uintptr_t)FileOffset)<br>
+ << " flags: " << flags<br>
+ << " SID: " << SectionID<br>
+ << " Offset: " << format("%p", SectOffset));<br>
+ if (isGlobal)<br>
+ SymbolTable[Name] = SymbolLoc(SectionID, SectOffset);<br>
+ }<br>
}<br>
DEBUG(dbgs() << "\tType: " << SymType << " Name: " << Name << "\n");<br>
}<br>
<br>
+ // Allocate common symbols<br>
+ if (CommonSize != 0)<br>
+ emitCommonSymbols(CommonSymbols, CommonSize, LocalSymbols);<br>
+<br>
// Parse and proccess relocations<br>
DEBUG(dbgs() << "Parse relocations:\n");<br>
for (section_iterator si = obj->begin_sections(),<br>
@@ -150,6 +167,38 @@<br>
return false;<br>
}<br>
<br>
+unsigned RuntimeDyldImpl::emitCommonSymbols(const CommonSymbolMap &Map,<br>
+ uint64_t TotalSize,<br>
+ LocalSymbolMap &LocalSymbols) {<br>
+ // Allocate memory for the section<br>
+ unsigned SectionID = Sections.size();<br>
+ uint8_t *Addr = MemMgr->allocateDataSection(TotalSize, sizeof(void*),<br>
+ SectionID);<br>
+ if (!Addr)<br>
+ report_fatal_error("Unable to allocate memory for common symbols!");<br>
+ uint64_t Offset = 0;<br>
+ Sections.push_back(SectionEntry(Addr, TotalSize, TotalSize, 0));<br>
+ memset(Addr, 0, TotalSize);<br>
+<br>
+ DEBUG(dbgs() << "emitCommonSection SectionID: " << SectionID<br>
+ << " new addr: " << format("%p", Addr)<br>
+ << " DataSize: " << TotalSize<br>
+ << "\n");<br>
+<br>
+ // Assign the address of each symbol<br>
+ for (CommonSymbolMap::const_iterator it = Map.begin(), itEnd = Map.end();<br>
+ it != itEnd; it++) {<br>
+ uint64_t Size = it->second;<br>
+ StringRef Name;<br>
+ it->first.getName(Name);<br>
+ LocalSymbols[Name.data()] = SymbolLoc(SectionID, Offset);<br>
+ Offset += Size;<br>
+ Addr += Size;<br>
+ }<br>
+<br>
+ return SectionID;<br>
+}<br>
+<br>
unsigned RuntimeDyldImpl::emitSection(const SectionRef &Section,<br>
bool IsCode) {<br>
<br>
@@ -158,7 +207,7 @@<br>
error_code err;<br>
if (StubSize > 0) {<br>
for (relocation_iterator i = Section.begin_relocations(),<br>
- e = Section.end_relocations(); i != e; i.increment(err))<br>
+ e = Section.end_relocations(); i != e; i.increment(err), Check(err))<br>
StubBufSize += StubSize;<br>
}<br>
StringRef data;<br>
@@ -167,22 +216,63 @@<br>
Check(Section.getAlignment(Alignment64));<br>
<br>
unsigned Alignment = (unsigned)Alignment64 & 0xffffffffL;<br>
- unsigned DataSize = data.size();<br>
- unsigned Allocate = DataSize + StubBufSize;<br>
+ bool IsRequired;<br>
+ bool IsVirtual;<br>
+ bool IsZeroInit;<br>
+ uint64_t DataSize;<br>
+ Check(Section.isRequiredForExecution(IsRequired));<br>
+ Check(Section.isVirtual(IsVirtual));<br>
+ Check(Section.isZeroInit(IsZeroInit));<br>
+ Check(Section.getSize(DataSize));<br>
+<br>
+ unsigned Allocate;<br>
unsigned SectionID = Sections.size();<br>
- const char *pData = data.data();<br>
- uint8_t *Addr = IsCode<br>
- ? MemMgr->allocateCodeSection(Allocate, Alignment, SectionID)<br>
- : MemMgr->allocateDataSection(Allocate, Alignment, SectionID);<br>
-<br>
- memcpy(Addr, pData, DataSize);<br>
- DEBUG(dbgs() << "emitSection SectionID: " << SectionID<br>
- << " obj addr: " << format("%p", pData)<br>
- << " new addr: " << format("%p", Addr)<br>
- << " DataSize: " << DataSize<br>
- << " StubBufSize: " << StubBufSize<br>
- << " Allocate: " << Allocate<br>
- << "\n");<br>
+ uint8_t *Addr;<br>
+ const char *pData = 0;<br>
+<br>
+ // Some sections, such as debug info, don't need to be loaded for execution.<br>
+ // Leave those where they are.<br>
+ if (IsRequired) {<br>
+ Allocate = DataSize + StubBufSize;<br>
+ Addr = IsCode<br>
+ ? MemMgr->allocateCodeSection(Allocate, Alignment, SectionID)<br>
+ : MemMgr->allocateDataSection(Allocate, Alignment, SectionID);<br>
+ if (!Addr)<br>
+ report_fatal_error("Unable to allocate section memory!");<br>
+<br>
+ // Virtual sections have no data in the object image, so leave pData = 0<br>
+ if (!IsVirtual)<br>
+ pData = data.data();<br>
+<br>
+ // Zero-initialize or copy the data from the image<br>
+ if (IsZeroInit || IsVirtual)<br>
+ memset(Addr, 0, DataSize);<br>
+ else<br>
+ memcpy(Addr, pData, DataSize);<br>
+<br>
+ DEBUG(dbgs() << "emitSection SectionID: " << SectionID<br>
+ << " obj addr: " << format("%p", pData)<br>
+ << " new addr: " << format("%p", Addr)<br>
+ << " DataSize: " << DataSize<br>
+ << " StubBufSize: " << StubBufSize<br>
+ << " Allocate: " << Allocate<br>
+ << "\n");<br>
+ }<br>
+ else {<br>
+ // Even if we didn't load the section, we need to record an entry for it<br>
+ // to handle later processing (and by 'handle' I mean don't do anything<br>
+ // with these sections).<br>
+ Allocate = 0;<br>
+ Addr = 0;<br>
+ DEBUG(dbgs() << "emitSection SectionID: " << SectionID<br>
+ << " obj addr: " << format("%p", data.data())<br>
+ << " new addr: 0"<br>
+ << " DataSize: " << DataSize<br>
+ << " StubBufSize: " << StubBufSize<br>
+ << " Allocate: " << Allocate<br>
+ << "\n");<br>
+ }<br>
+<br>
Sections.push_back(SectionEntry(Addr, Allocate, DataSize,(uintptr_t)pData));<br>
return SectionID;<br>
}<br>
@@ -259,15 +349,18 @@<br>
<br>
void RuntimeDyldImpl::resolveRelocationEntry(const RelocationEntry &RE,<br>
uint64_t Value) {<br>
- uint8_t *Target = Sections[RE.SectionID].Address + RE.Offset;<br>
- DEBUG(dbgs() << "\tSectionID: " << RE.SectionID<br>
- << " + " << RE.Offset << " (" << format("%p", Target) << ")"<br>
- << " Data: " << RE.Data<br>
- << " Addend: " << RE.Addend<br>
- << "\n");<br>
+ // Ignore relocations for sections that were not loaded<br>
+ if (Sections[RE.SectionID].Address != 0) {<br>
+ uint8_t *Target = Sections[RE.SectionID].Address + RE.Offset;<br>
+ DEBUG(dbgs() << "\tSectionID: " << RE.SectionID<br>
+ << " + " << RE.Offset << " (" << format("%p", Target) << ")"<br>
+ << " Data: " << RE.Data<br>
+ << " Addend: " << RE.Addend<br>
+ << "\n");<br>
<br>
- resolveRelocation(Target, Sections[RE.SectionID].LoadAddress + RE.Offset,<br>
- Value, RE.Data, RE.Addend);<br>
+ resolveRelocation(Target, Sections[RE.SectionID].LoadAddress + RE.Offset,<br>
+ Value, RE.Data, RE.Addend);<br>
+ }<br>
}<br>
<br>
void RuntimeDyldImpl::resolveRelocationList(const RelocationList &Relocs,<br>
<br>
Modified: llvm/trunk/lib/ExecutionEngine/RuntimeDyld/RuntimeDyldELF.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/ExecutionEngine/RuntimeDyld/RuntimeDyldELF.cpp?rev=154610&r1=154609&r2=154610&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/ExecutionEngine/RuntimeDyld/RuntimeDyldELF.cpp?rev=154610&r1=154609&r2=154610&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/ExecutionEngine/RuntimeDyld/RuntimeDyldELF.cpp (original)<br>
+++ llvm/trunk/lib/ExecutionEngine/RuntimeDyld/RuntimeDyldELF.cpp Thu Apr 12 15:13:57 2012<br>
@@ -71,7 +71,8 @@<br>
switch (Type) {<br>
case ELF::R_386_32: {<br>
uint32_t *Target = (uint32_t*)(LocalAddress);<br>
- *Target = Value + Addend;<br>
+ uint32_t Placeholder = *Target;<br>
+ *Target = Placeholder + Value + Addend;<br>
break;<br>
}<br>
case ELF::R_386_PC32: {<br>
<br>
Modified: llvm/trunk/lib/ExecutionEngine/RuntimeDyld/RuntimeDyldImpl.h<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/ExecutionEngine/RuntimeDyld/RuntimeDyldImpl.h?rev=154610&r1=154609&r2=154610&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/ExecutionEngine/RuntimeDyld/RuntimeDyldImpl.h?rev=154610&r1=154609&r2=154610&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/ExecutionEngine/RuntimeDyld/RuntimeDyldImpl.h (original)<br>
+++ llvm/trunk/lib/ExecutionEngine/RuntimeDyld/RuntimeDyldImpl.h Thu Apr 12 15:13:57 2012<br>
@@ -110,6 +110,9 @@<br>
StringMap<SymbolLoc> SymbolTable;<br>
typedef DenseMap<const char*, SymbolLoc> LocalSymbolMap;<br>
<br>
+ // Keep a map of common symbols to their sizes<br>
+ typedef std::map<SymbolRef, unsigned> CommonSymbolMap;<br>
+<br>
// For each symbol, keep a list of relocations based on it. Anytime<br>
// its address is reassigned (the JIT re-compiled the function, e.g.),<br>
// the relocations get re-resolved.<br>
@@ -149,6 +152,12 @@<br>
return (uint8_t*)Sections[SectionID].Address;<br>
}<br>
<br>
+ /// \brief Emits a section containing common symbols.<br>
+ /// \return SectionID.<br>
+ unsigned emitCommonSymbols(const CommonSymbolMap &Map,<br>
+ uint64_t TotalSize,<br>
+ LocalSymbolMap &Symbols);<br>
+<br>
/// \brief Emits section data from the object file to the MemoryManager.<br>
/// \param IsCode if it's true then allocateCodeSection() will be<br>
/// used for emmits, else allocateDataSection() will be used.<br>
<br>
Modified: llvm/trunk/lib/Object/COFFObjectFile.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Object/COFFObjectFile.cpp?rev=154610&r1=154609&r2=154610&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Object/COFFObjectFile.cpp?rev=154610&r1=154609&r2=154610&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/Object/COFFObjectFile.cpp (original)<br>
+++ llvm/trunk/lib/Object/COFFObjectFile.cpp Thu Apr 12 15:13:57 2012<br>
@@ -356,6 +356,27 @@<br>
return object_error::success;<br>
}<br>
<br>
+error_code COFFObjectFile::isSectionRequiredForExecution(DataRefImpl Sec,<br>
+ bool &Result) const {<br>
+ // FIXME: Unimplemented<br>
+ Result = true;<br>
+ return object_error::success;<br>
+}<br>
+<br>
+error_code COFFObjectFile::isSectionVirtual(DataRefImpl Sec,<br>
+ bool &Result) const {<br>
+ const coff_section *sec = toSec(Sec);<br>
+ Result = sec->Characteristics & COFF::IMAGE_SCN_CNT_UNINITIALIZED_DATA;<br>
+ return object_error::success;<br>
+}<br>
+<br>
+error_code COFFObjectFile::isSectionZeroInit(DataRefImpl Sec,<br>
+ bool &Result) const {<br>
+ // FIXME: Unimplemented<br>
+ Result = false;<br>
+ return object_error::success;<br>
+}<br>
+<br>
error_code COFFObjectFile::sectionContainsSymbol(DataRefImpl Sec,<br>
DataRefImpl Symb,<br>
bool &Result) const {<br>
<br>
Modified: llvm/trunk/lib/Object/MachOObjectFile.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Object/MachOObjectFile.cpp?rev=154610&r1=154609&r2=154610&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Object/MachOObjectFile.cpp?rev=154610&r1=154609&r2=154610&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/Object/MachOObjectFile.cpp (original)<br>
+++ llvm/trunk/lib/Object/MachOObjectFile.cpp Thu Apr 12 15:13:57 2012<br>
@@ -175,7 +175,12 @@<br>
BeginOffset = Entry->Value;<br>
SectionIndex = Entry->SectionIndex;<br>
if (!SectionIndex) {<br>
- Result = UnknownAddressOrSize;<br>
+ uint32_t flags = SymbolRef::SF_None;<br>
+ getSymbolFlags(DRI, flags);<br>
+ if (flags & SymbolRef::SF_Common)<br>
+ Result = Entry->Value;<br>
+ else<br>
+ Result = UnknownAddressOrSize;<br>
return object_error::success;<br>
}<br>
// Unfortunately symbols are unsorted so we need to touch all<br>
@@ -198,7 +203,12 @@<br>
BeginOffset = Entry->Value;<br>
SectionIndex = Entry->SectionIndex;<br>
if (!SectionIndex) {<br>
- Result = UnknownAddressOrSize;<br>
+ uint32_t flags = SymbolRef::SF_None;<br>
+ getSymbolFlags(DRI, flags);<br>
+ if (flags & SymbolRef::SF_Common)<br>
+ Result = Entry->Value;<br>
+ else<br>
+ Result = UnknownAddressOrSize;<br>
return object_error::success;<br>
}<br>
// Unfortunately symbols are unsorted so we need to touch all<br>
@@ -265,19 +275,22 @@<br>
uint32_t &Result) const {<br>
uint16_t MachOFlags;<br>
uint8_t MachOType;<br>
+ uint8_t MachOSectionIndex;<br>
if (MachOObj->is64Bit()) {<br>
InMemoryStruct<macho::Symbol64TableEntry> Entry;<br>
getSymbol64TableEntry(DRI, Entry);<br>
MachOFlags = Entry->Flags;<br>
MachOType = Entry->Type;<br>
+ MachOSectionIndex = Entry->SectionIndex;<br>
} else {<br>
InMemoryStruct<macho::SymbolTableEntry> Entry;<br>
getSymbolTableEntry(DRI, Entry);<br>
MachOFlags = Entry->Flags;<br>
MachOType = Entry->Type;<br>
+ MachOSectionIndex = Entry->SectionIndex;<br>
}<br>
<br>
- // TODO: Correctly set SF_ThreadLocal and SF_Common.<br>
+ // TODO: Correctly set SF_ThreadLocal<br>
Result = SymbolRef::SF_None;<br>
<br>
if ((MachOType & MachO::NlistMaskType) == MachO::NListTypeUndefined)<br>
@@ -286,8 +299,11 @@<br>
if (MachOFlags & macho::STF_StabsEntryMask)<br>
Result |= SymbolRef::SF_FormatSpecific;<br>
<br>
- if (MachOType & MachO::NlistMaskExternal)<br>
+ if (MachOType & MachO::NlistMaskExternal) {<br>
Result |= SymbolRef::SF_Global;<br>
+ if ((MachOType & MachO::NlistMaskType) == MachO::NListTypeUndefined)<br>
+ Result |= SymbolRef::SF_Common;<br>
+ }<br>
<br>
if (MachOFlags & (MachO::NListDescWeakRef | MachO::NListDescWeakDef))<br>
Result |= SymbolRef::SF_Weak;<br>
@@ -566,6 +582,37 @@<br>
return object_error::success;<br>
}<br>
<br>
+error_code MachOObjectFile::isSectionRequiredForExecution(DataRefImpl Sec,<br>
+ bool &Result) const {<br>
+ // FIXME: Unimplemented<br>
+ Result = true;<br>
+ return object_error::success;<br>
+}<br>
+<br>
+error_code MachOObjectFile::isSectionVirtual(DataRefImpl Sec,<br>
+ bool &Result) const {<br>
+ // FIXME: Unimplemented<br>
+ Result = false;<br>
+ return object_error::success;<br>
+}<br>
+<br>
+error_code MachOObjectFile::isSectionZeroInit(DataRefImpl DRI,<br>
+ bool &Result) const {<br>
+ if (MachOObj->is64Bit()) {<br>
+ InMemoryStruct<macho::Section64> Sect;<br>
+ getSection64(DRI, Sect);<br>
+ Result = (Sect->Flags & MachO::SectionTypeZeroFill ||<br>
+ Sect->Flags & MachO::SectionTypeZeroFillLarge);<br>
+ } else {<br>
+ InMemoryStruct<macho::Section> Sect;<br>
+ getSection(DRI, Sect);<br>
+ Result = (Sect->Flags & MachO::SectionTypeZeroFill ||<br>
+ Sect->Flags & MachO::SectionTypeZeroFillLarge);<br>
+ }<br>
+<br>
+ return object_error::success;<br>
+}<br>
+<br>
error_code MachOObjectFile::sectionContainsSymbol(DataRefImpl Sec,<br>
DataRefImpl Symb,<br>
bool &Result) const {<br>
<br>
Modified: llvm/trunk/test/ExecutionEngine/2002-12-16-ArgTest.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/2002-12-16-ArgTest.ll?rev=154610&r1=154609&r2=154610&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/2002-12-16-ArgTest.ll?rev=154610&r1=154609&r2=154610&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/ExecutionEngine/2002-12-16-ArgTest.ll (original)<br>
+++ llvm/trunk/test/ExecutionEngine/2002-12-16-ArgTest.ll Thu Apr 12 15:13:57 2012<br>
@@ -1,5 +1,4 @@<br>
; RUN: %lli %s > /dev/null<br>
-; XFAIL: mcjit<br>
<br>
@.LC0 = internal global [10 x i8] c"argc: %d\0A\00" ; <[10 x i8]*> [#uses=1]<br>
<br>
<br>
Modified: llvm/trunk/test/ExecutionEngine/2003-01-04-ArgumentBug.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/2003-01-04-ArgumentBug.ll?rev=154610&r1=154609&r2=154610&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/2003-01-04-ArgumentBug.ll?rev=154610&r1=154609&r2=154610&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/ExecutionEngine/2003-01-04-ArgumentBug.ll (original)<br>
+++ llvm/trunk/test/ExecutionEngine/2003-01-04-ArgumentBug.ll Thu Apr 12 15:13:57 2012<br>
@@ -1,6 +1,5 @@<br>
; RUN: %lli %s > /dev/null<br>
; XFAIL: arm<br>
-; XFAIL: mcjit<br>
<br>
define i32 @foo(i32 %X, i32 %Y, double %A) {<br>
%cond212 = fcmp une double %A, 1.000000e+00 ; <i1> [#uses=1]<br>
<br>
Modified: llvm/trunk/test/ExecutionEngine/2003-01-04-LoopTest.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/2003-01-04-LoopTest.ll?rev=154610&r1=154609&r2=154610&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/2003-01-04-LoopTest.ll?rev=154610&r1=154609&r2=154610&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/ExecutionEngine/2003-01-04-LoopTest.ll (original)<br>
+++ llvm/trunk/test/ExecutionEngine/2003-01-04-LoopTest.ll Thu Apr 12 15:13:57 2012<br>
@@ -1,6 +1,5 @@<br>
; RUN: %lli %s > /dev/null<br>
; XFAIL: arm<br>
-; XFAIL: mcjit<br>
<br>
define i32 @main() {<br>
call i32 @mylog( i32 4 ) ; <i32>:1 [#uses=0]<br>
<br>
Modified: llvm/trunk/test/ExecutionEngine/2003-05-06-LivenessClobber.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/2003-05-06-LivenessClobber.ll?rev=154610&r1=154609&r2=154610&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/2003-05-06-LivenessClobber.ll?rev=154610&r1=154609&r2=154610&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/ExecutionEngine/2003-05-06-LivenessClobber.ll (original)<br>
+++ llvm/trunk/test/ExecutionEngine/2003-05-06-LivenessClobber.ll Thu Apr 12 15:13:57 2012<br>
@@ -2,7 +2,6 @@<br>
;<br>
; RUN: not %lli %s<br>
; XFAIL: arm<br>
-; XFAIL: mcjit<br>
<br>
@test = global i64 0 ; <i64*> [#uses=1]<br>
<br>
<br>
Modified: llvm/trunk/test/ExecutionEngine/2003-05-07-ArgumentTest.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/2003-05-07-ArgumentTest.ll?rev=154610&r1=154609&r2=154610&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/2003-05-07-ArgumentTest.ll?rev=154610&r1=154609&r2=154610&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/ExecutionEngine/2003-05-07-ArgumentTest.ll (original)<br>
+++ llvm/trunk/test/ExecutionEngine/2003-05-07-ArgumentTest.ll Thu Apr 12 15:13:57 2012<br>
@@ -1,6 +1,5 @@<br>
; RUN: %lli %s test<br>
; XFAIL: arm<br>
-; XFAIL: mcjit<br>
<br>
declare i32 @puts(i8*)<br>
<br>
<br>
Modified: llvm/trunk/test/ExecutionEngine/2003-08-21-EnvironmentTest.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/2003-08-21-EnvironmentTest.ll?rev=154610&r1=154609&r2=154610&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/2003-08-21-EnvironmentTest.ll?rev=154610&r1=154609&r2=154610&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/ExecutionEngine/2003-08-21-EnvironmentTest.ll (original)<br>
+++ llvm/trunk/test/ExecutionEngine/2003-08-21-EnvironmentTest.ll Thu Apr 12 15:13:57 2012<br>
@@ -1,6 +1,5 @@<br>
; RUN: %lli %s > /dev/null<br>
; XFAIL: arm<br>
-; XFAIL: mcjit<br>
<br>
;<br>
; Regression Test: EnvironmentTest.ll<br>
<br>
Modified: llvm/trunk/test/ExecutionEngine/2005-12-02-TailCallBug.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/2005-12-02-TailCallBug.ll?rev=154610&r1=154609&r2=154610&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/2005-12-02-TailCallBug.ll?rev=154610&r1=154609&r2=154610&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/ExecutionEngine/2005-12-02-TailCallBug.ll (original)<br>
+++ llvm/trunk/test/ExecutionEngine/2005-12-02-TailCallBug.ll Thu Apr 12 15:13:57 2012<br>
@@ -1,7 +1,6 @@<br>
; PR672<br>
; RUN: %lli %s<br>
; XFAIL: arm<br>
-; XFAIL: mcjit-ia32<br>
<br>
define i32 @main() {<br>
%f = bitcast i32 (i32, i32*, i32)* @check_tail to i32* ; <i32*> [#uses=1]<br>
<br>
Modified: llvm/trunk/test/ExecutionEngine/hello.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/hello.ll?rev=154610&r1=154609&r2=154610&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/hello.ll?rev=154610&r1=154609&r2=154610&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/ExecutionEngine/hello.ll (original)<br>
+++ llvm/trunk/test/ExecutionEngine/hello.ll Thu Apr 12 15:13:57 2012<br>
@@ -1,6 +1,5 @@<br>
; RUN: %lli %s > /dev/null<br>
; XFAIL: arm<br>
-; XFAIL: mcjit<br>
<br>
@.LC0 = internal global [12 x i8] c"Hello World\00" ; <[12 x i8]*> [#uses=1]<br>
<br>
<br>
Modified: llvm/trunk/test/ExecutionEngine/hello2.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/hello2.ll?rev=154610&r1=154609&r2=154610&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/hello2.ll?rev=154610&r1=154609&r2=154610&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/ExecutionEngine/hello2.ll (original)<br>
+++ llvm/trunk/test/ExecutionEngine/hello2.ll Thu Apr 12 15:13:57 2012<br>
@@ -1,6 +1,5 @@<br>
; RUN: %lli %s > /dev/null<br>
; XFAIL: arm<br>
-; XFAIL: mcjit<br>
<br>
@X = global i32 7 ; <i32*> [#uses=0]<br>
@msg = internal global [13 x i8] c"Hello World\0A\00" ; <[13 x i8]*> [#uses=1]<br>
<br>
Modified: llvm/trunk/test/ExecutionEngine/stubs.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/stubs.ll?rev=154610&r1=154609&r2=154610&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/stubs.ll?rev=154610&r1=154609&r2=154610&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/ExecutionEngine/stubs.ll (original)<br>
+++ llvm/trunk/test/ExecutionEngine/stubs.ll Thu Apr 12 15:13:57 2012<br>
@@ -1,6 +1,5 @@<br>
; RUN: %lli -disable-lazy-compilation=false %s<br>
; XFAIL: arm<br>
-; XFAIL: mcjit<br>
<br>
define i32 @main() nounwind {<br>
entry:<br>
<br>
Modified: llvm/trunk/test/ExecutionEngine/test-call-no-external-funcs.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/test-call-no-external-funcs.ll?rev=154610&r1=154609&r2=154610&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/test-call-no-external-funcs.ll?rev=154610&r1=154609&r2=154610&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/ExecutionEngine/test-call-no-external-funcs.ll (original)<br>
+++ llvm/trunk/test/ExecutionEngine/test-call-no-external-funcs.ll Thu Apr 12 15:13:57 2012<br>
@@ -1,6 +1,5 @@<br>
; RUN: %lli %s > /dev/null<br>
; XFAIL: arm<br>
-; XFAIL: mcjit<br>
<br>
define i32 @_Z14func_exit_codev() nounwind uwtable {<br>
entry:<br>
<br>
Modified: llvm/trunk/test/ExecutionEngine/test-call.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/test-call.ll?rev=154610&r1=154609&r2=154610&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/test-call.ll?rev=154610&r1=154609&r2=154610&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/ExecutionEngine/test-call.ll (original)<br>
+++ llvm/trunk/test/ExecutionEngine/test-call.ll Thu Apr 12 15:13:57 2012<br>
@@ -1,6 +1,5 @@<br>
; RUN: %lli %s > /dev/null<br>
; XFAIL: arm<br>
-; XFAIL: mcjit<br>
<br>
declare void @exit(i32)<br>
<br>
<br>
Modified: llvm/trunk/test/ExecutionEngine/test-common-symbols.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/test-common-symbols.ll?rev=154610&r1=154609&r2=154610&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/test-common-symbols.ll?rev=154610&r1=154609&r2=154610&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/ExecutionEngine/test-common-symbols.ll (original)<br>
+++ llvm/trunk/test/ExecutionEngine/test-common-symbols.ll Thu Apr 12 15:13:57 2012<br>
@@ -1,6 +1,5 @@<br>
; RUN: %lli -O0 -disable-lazy-compilation=false %s<br>
; XFAIL: arm<br>
-; XFAIL: mcjit<br>
<br>
; The intention of this test is to verify that symbols mapped to COMMON in ELF<br>
; work as expected.<br>
<br>
Modified: llvm/trunk/test/ExecutionEngine/test-fp-no-external-funcs.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/test-fp-no-external-funcs.ll?rev=154610&r1=154609&r2=154610&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/test-fp-no-external-funcs.ll?rev=154610&r1=154609&r2=154610&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/ExecutionEngine/test-fp-no-external-funcs.ll (original)<br>
+++ llvm/trunk/test/ExecutionEngine/test-fp-no-external-funcs.ll Thu Apr 12 15:13:57 2012<br>
@@ -1,5 +1,4 @@<br>
; RUN: %lli %s > /dev/null<br>
-; XFAIL: mcjit<br>
<br>
define double @test(double* %DP, double %Arg) {<br>
%D = load double* %DP ; <double> [#uses=1]<br>
<br>
Modified: llvm/trunk/test/ExecutionEngine/test-fp.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/test-fp.ll?rev=154610&r1=154609&r2=154610&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/test-fp.ll?rev=154610&r1=154609&r2=154610&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/ExecutionEngine/test-fp.ll (original)<br>
+++ llvm/trunk/test/ExecutionEngine/test-fp.ll Thu Apr 12 15:13:57 2012<br>
@@ -1,5 +1,4 @@<br>
; RUN: %lli %s > /dev/null<br>
-; XFAIL: mcjit<br>
<br>
define double @test(double* %DP, double %Arg) {<br>
%D = load double* %DP ; <double> [#uses=1]<br>
<br>
Modified: llvm/trunk/test/ExecutionEngine/test-global-init-nonzero.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/test-global-init-nonzero.ll?rev=154610&r1=154609&r2=154610&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/test-global-init-nonzero.ll?rev=154610&r1=154609&r2=154610&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/ExecutionEngine/test-global-init-nonzero.ll (original)<br>
+++ llvm/trunk/test/ExecutionEngine/test-global-init-nonzero.ll Thu Apr 12 15:13:57 2012<br>
@@ -1,6 +1,5 @@<br>
; RUN: %lli %s > /dev/null<br>
; XFAIL: arm<br>
-; XFAIL: mcjit<br>
<br>
@count = global i32 1, align 4<br>
<br>
<br>
Modified: llvm/trunk/test/ExecutionEngine/test-global.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/test-global.ll?rev=154610&r1=154609&r2=154610&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/test-global.ll?rev=154610&r1=154609&r2=154610&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/ExecutionEngine/test-global.ll (original)<br>
+++ llvm/trunk/test/ExecutionEngine/test-global.ll Thu Apr 12 15:13:57 2012<br>
@@ -1,6 +1,5 @@<br>
; RUN: %lli %s > /dev/null<br>
; XFAIL: arm<br>
-; XFAIL: mcjit<br>
<br>
@count = global i32 0, align 4<br>
<br>
<br>
Modified: llvm/trunk/test/ExecutionEngine/test-loadstore.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/test-loadstore.ll?rev=154610&r1=154609&r2=154610&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/test-loadstore.ll?rev=154610&r1=154609&r2=154610&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/ExecutionEngine/test-loadstore.ll (original)<br>
+++ llvm/trunk/test/ExecutionEngine/test-loadstore.ll Thu Apr 12 15:13:57 2012<br>
@@ -1,6 +1,5 @@<br>
; RUN: %lli %s > /dev/null<br>
; XFAIL: arm<br>
-; XFAIL: mcjit-ia32<br>
<br>
define void @test(i8* %P, i16* %P.upgrd.1, i32* %P.upgrd.2, i64* %P.upgrd.3) {<br>
%V = load i8* %P ; <i8> [#uses=1]<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br><br clear="all"><br>-- <br>~Craig<br>