<p dir="ltr">I think less build system complexity in both CMake and autoconf is better. The best would be to delete this built-in altogether. :-)</p>
<p dir="ltr">Sent from phone</p>
<div class="gmail_quote">On Sep 5, 2014 7:04 PM, "Saleem Abdulrasool" <<a href="mailto:compnerd@compnerd.org">compnerd@compnerd.org</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">On Fri, Sep 5, 2014 at 4:37 PM, Stephen Hines <span dir="ltr"><<a href="mailto:srhines@google.com" target="_blank">srhines@google.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Does this look better?</div></div></blockquote><div><br></div><div>Depends on how you squint :-).  I find it ugly, but correct.</div><div><br></div><div>There is a patch that I have on phab that adds support for the Makefile based build system for building compiler-rt for Windows x86, x86_64, and ARM.  This file is entirely elided, which you have done.  I think dropping the file in the build system is nicer, but if that doesn't work, then this is fine.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Thanks,</div><div>Steve</div><div><br></div><div>From: Stephen Hines <<a href="mailto:srhines@google.com" target="_blank">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 style="white-space:pre-wrap"> </span>size_t length = endPage - startPage;</div><div> <span style="white-space:pre-wrap">   </span>(void) mprotect((void *)startPage, length, PROT_READ | PROT_WRITE | PROT_EXEC);</div><div> }</div><div>+</div><div>+#endif</div><span><font color="#888888"><div>-- </div><div>2.1.0.rc2.206.gedb03e5</div><div><br></div></font></span></div><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>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><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>
</div></div><br>_______________________________________________<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><br clear="all"><div><br></div>-- <br>Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org
</div></div>
</blockquote></div>