<div dir="ltr">Hi Philip,<div><br></div><div>No - as far as I know there's no bug for this. Please go ahead and file one. If we add a simple custom memory manager to llvm-rtdyld that's compatible with the small memory model (maybe just reserve ~1Mb up front? We'll never need more than that for regression tests). Then we can add a flag to chose that memory manager and tests specifically for small-code-model relocations.</div><div><br></div><div>Cheers,</div><div>Lang.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Apr 9, 2015 at 8:50 AM, Philip Reames <span dir="ltr"><<a href="mailto:listmail@philipreames.com" target="_blank">listmail@philipreames.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Lang,<br>
<br>
Is there a bug open for getting a small model memory manager implemented and these tests restored?  I'd like to be able to track the work on this.  If you're not going to get to it soon, I might send a patch myself.  Supporting a small memory model is something which is useful to me and I'm nervous that it's currently untested in tree.<span class="HOEnZb"><font color="#888888"><br>
<br>
Philip</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
On 03/31/2015 11:19 AM, Lang Hames wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Author: lhames<br>
Date: Tue Mar 31 13:19:25 2015<br>
New Revision: 233722<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=233722&view=rev" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project?rev=233722&view=rev</a><br>
Log:<br>
[Orc][MCJIT] Remove the small code model regression tests.<br>
<br>
These regression tests are supposed to test small code model support, but have<br>
been XFAIL'd because we don't have an in-tree memory manager that can guarantee<br>
a small-code-model compatible memory layout. Unfortunately, they can<br>
occasionally pass if they get lucky with memory allocation, causing unexpected<br>
passes on the bots. That's not very helpful.<br>
<br>
I'm going to remove these until we have the infrastructure (small-code-model<br>
compatible memory manager) to run them properly.<br>
<br>
<br>
Removed:<br>
     llvm/trunk/test/<u></u>ExecutionEngine/MCJIT/hello-<u></u>sm-pic.ll<br>
     llvm/trunk/test/<u></u>ExecutionEngine/OrcMCJIT/<u></u>hello-sm-pic.ll<br>
<br>
Removed: llvm/trunk/test/<u></u>ExecutionEngine/MCJIT/hello-<u></u>sm-pic.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/MCJIT/hello-sm-pic.ll?rev=233721&view=auto" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project/llvm/trunk/test/<u></u>ExecutionEngine/MCJIT/hello-<u></u>sm-pic.ll?rev=233721&view=auto</a><br>
==============================<u></u>==============================<u></u>==================<br>
--- llvm/trunk/test/<u></u>ExecutionEngine/MCJIT/hello-<u></u>sm-pic.ll (original)<br>
+++ llvm/trunk/test/<u></u>ExecutionEngine/MCJIT/hello-<u></u>sm-pic.ll (removed)<br>
@@ -1,12 +0,0 @@<br>
-; RUN: %lli -relocation-model=pic -code-model=small %s > /dev/null<br>
-; XFAIL: mips, i686, i386, darwin, aarch64, arm<br>
-<br>
-@.LC0 = internal global [12 x i8] c"Hello World\00"            ; <[12 x i8]*> [#uses=1]<br>
-<br>
-declare i32 @puts(i8*)<br>
-<br>
-define i32 @main() {<br>
-       %reg210 = call i32 @puts( i8* getelementptr ([12 x i8], [12 x i8]* @.LC0, i64 0, i64 0) )               ; <i32> [#uses=0]<br>
-       ret i32 0<br>
-}<br>
-<br>
<br>
Removed: llvm/trunk/test/<u></u>ExecutionEngine/OrcMCJIT/<u></u>hello-sm-pic.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/OrcMCJIT/hello-sm-pic.ll?rev=233721&view=auto" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project/llvm/trunk/test/<u></u>ExecutionEngine/OrcMCJIT/<u></u>hello-sm-pic.ll?rev=233721&<u></u>view=auto</a><br>
==============================<u></u>==============================<u></u>==================<br>
--- llvm/trunk/test/<u></u>ExecutionEngine/OrcMCJIT/<u></u>hello-sm-pic.ll (original)<br>
+++ llvm/trunk/test/<u></u>ExecutionEngine/OrcMCJIT/<u></u>hello-sm-pic.ll (removed)<br>
@@ -1,12 +0,0 @@<br>
-; RUN: %lli -jit-kind=orc-mcjit -relocation-model=pic -code-model=small %s > /dev/null<br>
-; XFAIL: mips, i686, i386, darwin, aarch64, arm<br>
-<br>
-@.LC0 = internal global [12 x i8] c"Hello World\00"            ; <[12 x i8]*> [#uses=1]<br>
-<br>
-declare i32 @puts(i8*)<br>
-<br>
-define i32 @main() {<br>
-       %reg210 = call i32 @puts( i8* getelementptr ([12 x i8], [12 x i8]* @.LC0, i64 0, i64 0) )               ; <i32> [#uses=0]<br>
-       ret i32 0<br>
-}<br>
-<br>
<br>
<br>
______________________________<u></u>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">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/<u></u>mailman/listinfo/llvm-commits</a><br>
</blockquote>
<br>
</div></div></blockquote></div><br></div>