<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Feb 3, 2015 at 4:07 AM, Renato Golin <span dir="ltr"><<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On 30 January 2015 at 20:43, Jonathan Roelofs <<a href="mailto:jonathan@codesourcery.com" target="_blank">jonathan@codesourcery.com</a>> wrote:<br>
</span><span>> Last time we brought this up, there was only partial consensus, and then<br>
> someone arbitrarily declared total consensus (without compelling arguments<br>
> in any particular direction) that we were going to move it to compiler_rt.<br>
> Then the discussion fell on the floor because no-one had time to actually do<br>
> the move. Please, let's not let that happen again this time.<br>
<br>
</span>So, do we have a consensus?<br>
<br>
AFAICS, the most agree solution (with optionals to be defined): <br></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">1. Move Unwinder to its own repository in the LLVM server<br></blockquote><div><br></div><div>I think that we have consensus here, and we've given people time to chime in.  What exactly is the process for making this happen?  I assume that aKor or someone else would need to create the repository on the server side so that it can be populated or is there someone else who should be contacted so that we can get this going?</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
2. Make the CMake connections from libc++abi and compiler-rt<br>
  2.1 OPTIONAL 1: err if libunwinder is not there, clang errs if rtlib=RT<br>
  2.2 OPTIONAL 2: warns if libunwind is not there, clang errs if rtlib=RT<br>
  2.3 OPTIONAL 3: nothing, make clang smarter to pick existing unwinder<br>
3. Change clang to assume -lunwind when --rtlib=compiler-rt<br>
  3.1 OPTIONAL 4: allow linker error if no -lunwind / -lgcc_s<br>
  3.2 OPTIONAL 5: Add option to change unwinder library by not adding<br>
-lunwind/-lgcc_s, but whatever comes as argument<br>
<br>
1, 2, and 3 must be changed.<br>
<br>
I vote for adding { 2.2, 3.1 } for now, { 2.2, 3.1, 3.2 } for later.<br>
<br>
My idea for 3.2 is something like --unwinder=libgcc_s / libunwind, or<br>
something like that.<br>
<br>
I personally don't think the front-end scanning existing libraries is<br>
a good thing to do, but I'm not against the idea, if anyone feels<br>
strongly about it.<br>
<br>
If all of us could agree to a common solution, and make sure all<br>
interested parties are in, we should do the move before 3.7.<br>
<br>
Please, cast your votes.<br>
<br>
cheers,<br>
--renato<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org</div>
</div></div>