<div dir="ltr"><div>I wonder if we should just rely on the ImportAddressTableRVA instead of first trying to use the ImportLookupTableRVA.<br></div><div><br></div><div>What are your thoughts?</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jul 4, 2016 at 11:05 AM, 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Cituji David Majnemer <<a href="mailto:david.majnemer@gmail.com" target="_blank">david.majnemer@gmail.com</a>>:<span class=""><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Please clang-format the changes in your patch.<br>
</blockquote>
<br></span>
Not sure what exactly this means. Found and used this: <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> then created a new patch<span class=""><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Also, LLVM does not use else after a return: <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>
</blockquote>
<br></span>
Hope this is better. New patch as attachement.<br>
<br>
</blockquote></div><br></div>