<div dir="ltr">(re-adding the list)<div><br></div><div>Rui: ping?<div><div><br></div><div>Peter</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Oct 14, 2016 at 2:48 AM, Eugene Leviant <span dir="ltr"><<a href="mailto:evgeny.leviant@gmail.com" target="_blank">evgeny.leviant@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I've just applied your patch and it looks good at a first glance. That<br>
said, I don't have any objections against it.<br>
<div class="HOEnZb"><div class="h5"><br>
2016-10-14 12:04 GMT+03:00 Peter Collingbourne <<a href="mailto:peter@pcc.me.uk">peter@pcc.me.uk</a>>:<br>
> It looks like the previous behavior (before your change) was to create the<br>
> symbols unconditionally which caused a duplicate symbol error with linker<br>
> scripts and your change prevented the error by introducing the addSynthetic<br>
> function which would define them only if they did not already exist.<br>
><br>
> As I mentioned in the commit message, this is almost exactly what<br>
> addOptionalSymbol is already doing, so there's no real functional change<br>
> here.<br>
><br>
> Peter<br>
><br>
><br>
> On Oct 14, 2016 01:31, "Eugene Leviant" <<a href="mailto:evgeny.leviant@gmail.com">evgeny.leviant@gmail.com</a>> wrote:<br>
>><br>
>> evgeny777 added a comment.<br>
>><br>
>> addSynthetic() was introduced to allow defining start/end symbols in<br>
>> linker scripts<br>
>><br>
>> Please take a look at<br>
>> <a href="https://reviews.llvm.org/D23405" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D23405</a><br>
>><br>
>><br>
>> <a href="https://reviews.llvm.org/D25545" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D25545</a><br>
>><br>
>><br>
>><br>
><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">-- <div>Peter</div></div></div>
</div></div></div></div>