<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jul 4, 2016 at 9:52 PM, David Majnemer <span dir="ltr"><<a href="mailto:david.majnemer@gmail.com" target="_blank">david.majnemer@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 dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On Mon, Jul 4, 2016 at 2:10 PM, Bandzi Michal <span dir="ltr"><<a href="mailto:xbandz00@stud.fit.vutbr.cz" target="_blank">xbandz00@stud.fit.vutbr.cz</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">Unless I made some logical error I can't see, it should work that way, doesn't it?<br>
<br>
if (lookupTable)<br>
  return func(lookupTable, ...)<br>
return func(addressTable, ...)</blockquote><div><br></div></span><div>I was trying to suggest that we ignore the lookup table case and just do:</div><span class=""><div>  return func(addressTable, ...)<br></div></span></div></div></div></blockquote><div><br></div><div><br></div><div>Thinking about it some more... Let's just have two sets of functions, one for the lookupTable and one for the addressTable.</div><div>Let's let the caller decide what information they are interested in :)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class=""><div></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div><div><br>
<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
I wonder if we should just rely on the ImportAddressTableRVA instead of<br>
first trying to use the ImportLookupTableRVA.<br>
<br>
What are your thoughts?<br>
<br>
On Mon, Jul 4, 2016 at 11:05 AM, Bandzi Michal <<a href="mailto:xbandz00@stud.fit.vutbr.cz" target="_blank">xbandz00@stud.fit.vutbr.cz</a>><br>
wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
Cituji David Majnemer <<a href="mailto:david.majnemer@gmail.com" target="_blank">david.majnemer@gmail.com</a>>:<br>
<br>
Please clang-format the changes in your patch.<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
<br>
</blockquote>
<br>
Not sure what exactly this means. Found and used this:<br>
<a href="http://clang.llvm.org/docs/ClangFormat.html#script-for-patch-reformatting" rel="noreferrer" target="_blank">http://clang.llvm.org/docs/ClangFormat.html#script-for-patch-reformatting</a><br>
then created a new patch<br>
<br>
Also, LLVM does not use else after a return:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
<a href="http://llvm.org/docs/CodingStandards.html#don-t-use-else-after-a-return" rel="noreferrer" target="_blank">http://llvm.org/docs/CodingStandards.html#don-t-use-else-after-a-return</a><br>
<br>
</blockquote>
<br>
Hope this is better. New patch as attachement.<br>
<br>
<br>
</blockquote>
<br>
</blockquote>
<br>
<br>
<br>
</div></div></blockquote></span></div><br></div></div>
</blockquote></div><br></div></div>