<div class="gmail_quote">On Fri, Jan 13, 2012 at 1:00 AM, Evgeniy Stepanov <span dir="ltr"><<a href="mailto:eugeni.stepanov@gmail.com">eugeni.stepanov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<br>
<br>
this patch removes --hash-style option from driver-generated link<br>
command line on Android. The reason is that Android linker does not<br>
support .gnu.hash, so the default (sysv) is the correct value. What's<br>
worse, there is a bug in the older version of GNU ld used in the<br>
current Android toolchain that messes up .hash section whenever<br>
.gnu.hash is present.<br></blockquote><div><br></div><div>Oof.</div><div><br></div><div>This generally looks fine, feel free to commit. I would move the comment you added to be with the comment about MIPS not supporting this construct. They're both comments on the condition block, not on the IsAndroid or IsMips variables. The variables are pretty clear. =D</div>
</div>