<p dir="ltr">It looks like the previous behavior (before your change) was to create the symbols unconditionally which caused a duplicate symbol error with linker scripts and your change prevented the error by introducing the addSynthetic function which would define them only if they did not already exist.</p>
<p dir="ltr">As I mentioned in the commit message, this is almost exactly what addOptionalSymbol is already doing, so there's no real functional change here.</p>
<p dir="ltr">Peter</p>
<div class="gmail_extra"><br><div class="gmail_quote">On Oct 14, 2016 01:31, "Eugene Leviant" <<a href="mailto:evgeny.leviant@gmail.com">evgeny.leviant@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">evgeny777 added a comment.<br>
<br>
addSynthetic() was introduced to allow defining start/end symbols in 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>
</blockquote></div></div>