<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<meta name="Generator" content="Microsoft Word 14 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
{font-family:Tahoma;
panose-1:2 11 6 4 3 5 4 4 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0cm;
margin-bottom:.0001pt;
font-size:12.0pt;
font-family:"Times New Roman","serif";}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
{mso-style-priority:99;
color:purple;
text-decoration:underline;}
span.EmailStyle17
{mso-style-type:personal-reply;
font-family:"Calibri","sans-serif";
color:windowtext;}
.MsoChpDefault
{mso-style-type:export-only;
font-family:"Calibri","sans-serif";
mso-fareast-language:EN-US;}
@page WordSection1
{size:612.0pt 792.0pt;
margin:72.0pt 72.0pt 72.0pt 72.0pt;}
div.WordSection1
{page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="EN-GB" link="blue" vlink="purple">
<div class="WordSection1">
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif"">I've skimmed through RuntimeDyldCOFFX86_64 and there's a few places where it neglects to account for differences between host endian and target endian. If you search for
'(uint' you'll find them. It looks like COFF needs to use writeBytesUnaligned() like RuntimeDyldELF.cpp does.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif""><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif"">Hope that helps.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif""><o:p> </o:p></span></p>
<div style="border:none;border-left:solid blue 1.5pt;padding:0cm 0cm 0cm 4.0pt">
<div>
<div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0cm 0cm 0cm">
<p class="MsoNormal"><b><span lang="EN-US" style="font-size:10.0pt;font-family:"Tahoma","sans-serif"">From:</span></b><span lang="EN-US" style="font-size:10.0pt;font-family:"Tahoma","sans-serif""> llvm-commits [mailto:llvm-commits-bounces@lists.llvm.org]
<b>On Behalf Of </b>David Blaikie via llvm-commits<br>
<b>Sent:</b> 16 October 2015 22:50<br>
<b>To:</b> Lang Hames<br>
<b>Cc:</b> llvm-commits<br>
<b>Subject:</b> Re: [llvm] r250376 - [RuntimeDyld] Drop the '.s' suffix off the COFF test case - the MIPS bot started<o:p></o:p></span></p>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<p class="MsoNormal">On Thu, Oct 15, 2015 at 12:16 AM, Lang Hames via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<o:p></o:p></p>
<p class="MsoNormal">Author: lhames<br>
Date: Thu Oct 15 02:16:40 2015<br>
New Revision: 250376<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=250376&view=rev" target="_blank">
http://llvm.org/viewvc/llvm-project?rev=250376&view=rev</a><br>
Log:<br>
[RuntimeDyld] Drop the '.s' suffix off the COFF test case - the MIPS bot started<br>
failing when the suffix was added.<br>
<br>
I assume the lack of a '.s' suffix means that the test case just wasn't running<br>
before,<o:p></o:p></p>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Yep, only certain suffixes are actually run as tests, in llvm/test/lit.cfg:<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<p class="MsoNormal"># suffixes: A list of file extensions to treat as test files. This is overriden<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"># by individual lit.local.cfg files in the test subdirectories.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">config.suffixes = ['.ll', '.c', '.cxx', '.test', '.txt', '.s']<o:p></o:p></p>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal"> <o:p></o:p></p>
</div>
<blockquote style="border:none;border-left:solid #CCCCCC 1.0pt;padding:0cm 0cm 0cm 6.0pt;margin-left:4.8pt;margin-right:0cm">
<p class="MsoNormal">and it has never worked on MIPS. I'll investigate that tomorrow.<br>
<br>
<br>
Added:<br>
llvm/trunk/test/ExecutionEngine/RuntimeDyld/X86/COFF_x86_64<br>
- copied unchanged from r250371, llvm/trunk/test/ExecutionEngine/RuntimeDyld/X86/COFF_x86_64.s<br>
Removed:<br>
llvm/trunk/test/ExecutionEngine/RuntimeDyld/X86/COFF_x86_64.s<br>
<br>
Removed: llvm/trunk/test/ExecutionEngine/RuntimeDyld/X86/COFF_x86_64.s<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/RuntimeDyld/X86/COFF_x86_64.s?rev=250375&view=auto" target="_blank">
http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/RuntimeDyld/X86/COFF_x86_64.s?rev=250375&view=auto</a><br>
==============================================================================<br>
--- llvm/trunk/test/ExecutionEngine/RuntimeDyld/X86/COFF_x86_64.s (original)<br>
+++ llvm/trunk/test/ExecutionEngine/RuntimeDyld/X86/COFF_x86_64.s (removed)<br>
@@ -1,34 +0,0 @@<br>
-# RUN: llvm-mc -triple=x86_64-pc-win32 -filetype=obj -o %T/COFF_x86_64.o %s<br>
-# RUN: llvm-rtdyld -triple=x86_64-pc-win32 -verify -check=%s %/T/COFF_x86_64.o<br>
- .text<br>
- .def F;<br>
- .scl 2;<br>
- .type 32;<br>
- .endef<br>
- .globl __real400921f9f01b866e<br>
- .section .rdata,"dr",discard,__real400921f9f01b866e<br>
- .align 8<br>
-__real400921f9f01b866e:<br>
- .quad 4614256650576692846 # double 3.1415899999999999<br>
- .text<br>
- .globl F<br>
- .global inst1<br>
- .align 16, 0x90<br>
-F: # @F<br>
-.Ltmp0:<br>
-.seh_proc F<br>
-# BB#0: # %entry<br>
-.Ltmp1:<br>
- .seh_endprologue<br>
-# rtdyld-check: decode_operand(inst1, 4) = __real400921f9f01b866e - next_pc(inst1)<br>
-inst1:<br>
- movsd __real400921f9f01b866e(%rip), %xmm0 # xmm0 = mem[0],zero<br>
- retq<br>
-.Leh_func_end0:<br>
-.Ltmp2:<br>
- .seh_endproc<br>
-<br>
-# Make sure the JIT doesn't bail out on BSS sections.<br>
- .bss<br>
-bss_check:<br>
- .fill 8, 1, 0<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><o:p></o:p></p>
</blockquote>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
</div>
</div>
</div>
</body>
</html>