<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">On Tue, Jun 4, 2013 at 9:20 PM, Eric Christopher <span dir="ltr"><<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">> -    if (error(si->getSize(SymbolSize)) || SymbolSize == UnknownAddressOrSize)<br>
> +    // Getting symbol size is linear for Mach-O files, so avoid it.<br>
> +    if (isa<MachOObjectFile>(Obj))<br>
> +      SymbolSize = 0;<br>
> +    else if (error(si->getSize(SymbolSize)) ||<br>
> +             SymbolSize == UnknownAddressOrSize)<br>
<br>
</div>Expanding the comment here with the assumption that you're making<br>
below would be good. Otherwise the code doesn't make a whole lot of<br>
sense.<br></blockquote><div><br></div><div style>Yeah =/. I've tried to rewrite it in a more clear way in r183526, thanks!</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="HOEnZb"><div class="h5"><br>
> +  // Assume that symbols with zero size are large enough.<br>
> +  if (it->first.Addr < it->first.AddrEnd &&<br>
> +      it->first.AddrEnd <= Address)<br>
>      return false;<br>
>    Name = it->second.str();<br>
>    Addr = it->first.Addr;<br>
<br>
</div></div><span class="HOEnZb"><font color="#888888">-eric<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Alexey Samsonov, MSK</div>
</div></div>