<div dir="ltr"><div>Does this look better?</div><div><br></div><div>Thanks,</div><div>Steve</div><div><br></div><div>From: Stephen Hines <<a href="mailto:srhines@google.com">srhines@google.com</a>></div><div>Date: Tue, 2 Sep 2014 14:34:58 -0700</div><div>Subject: [PATCH] Remove enable_execute_stack functionality on Windows.</div><div><br></div><div>---</div><div> lib/builtins/enable_execute_stack.c | 3 +++</div><div> 1 file changed, 3 insertions(+)</div><div><br></div><div>diff --git a/lib/builtins/enable_execute_stack.c b/lib/builtins/enable_execute_stack.c</div><div>index 63d836b..c02be2a 100644</div><div>--- a/lib/builtins/enable_execute_stack.c</div><div>+++ b/lib/builtins/enable_execute_stack.c</div><div>@@ -8,6 +8,7 @@</div><div>  * ===----------------------------------------------------------------------===</div><div>  */</div><div> </div><div>+#if !defined(_WIN32)</div><div> #include "int_lib.h"</div><div> </div><div> #include <sys/mman.h></div><div>@@ -56,3 +57,5 @@ __enable_execute_stack(void* addr)</div><div> <span class="" style="white-space:pre">        </span>size_t length = endPage - startPage;</div><div> <span class="" style="white-space:pre">     </span>(void) mprotect((void *)startPage, length, PROT_READ | PROT_WRITE | PROT_EXEC);</div><div> }</div><div>+</div><div>+#endif</div><div>-- </div><div>2.1.0.rc2.206.gedb03e5</div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Sep 5, 2014 at 1:36 PM, Reid Kleckner <span dir="ltr"><<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Wow. It looks like LLVM doesn't reference this builtin. GCC however references it for nested function trampolines. It's not clear to me if we need to support that.<div><br></div><div>In the short term, I would just ifdef out the whole file on Windows. Feel free to just commit that if it makes your build work. In the longer term, I'd like to know if we can just delete this builtin.</div></div><div class="gmail_extra"><br><br><div class="gmail_quote"><div><div class="h5">On Fri, Sep 5, 2014 at 1:13 PM, Stephen Hines <span dir="ltr"><<a href="mailto:srhines@google.com" target="_blank">srhines@google.com</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><div dir="ltr">Ping.</div><div><div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Sep 2, 2014 at 2:41 PM, Stephen Hines <span dir="ltr"><<a href="mailto:srhines@google.com" target="_blank">srhines@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">The following patch allows compiler-rt to build/execute under a mingw build for windows. The existing code (ifdef-ed out on windows) doesn't build or execute correctly for this environment. Please take a look and let me know if it is ok to commit it.<div>
<br></div><div>Thanks,</div><div>Steve</div></div>
</blockquote></div><br></div>
</div></div><br></div></div>_______________________________________________<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/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>
</blockquote></div><br></div>