<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">2014-04-28 20:38 GMT+04:00 Rafael Ávila de Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span>:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div><div>Getting close. LGTM with a comment about why we can't use<br>

isPIEDefault: It would enable PIE only logic, like which TLS model to<br>
use, when compiling. That actually looks like a bug in the driver, but<br>
can be fixed in another commit.<br></div></div></blockquote><div>I don't understand how it could be fixed.</div><div>When compiler generates object it has no information regarding further usage of object: executable or library.</div>

<div>Function isPIEDefault turns on PIE for every object for specific target.</div><div>So it seems to be useful only for targets with no shared libraries.</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<div><div>
<br>
Cheers,<br>
Rafael<br>
<br>
</div></div><a href="http://reviews.llvm.org/D2668" target="_blank">http://reviews.llvm.org/D2668</a><br>
<br>
<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr">Alexey Volkov<div>Intel Corporation</div></div>
</div></div>