<div dir="ltr">Hmm, thinking more about this patch, this requires you to know whether a given object file is going to be linked into a DLL when building the C++ source so that you can pass -LD to clang-cl.  That may be not easy to get right depending on the build system, and when things go wrong you might need nasty -nodefaultlib arguments to the linker.  Maybe this is not the right approach after all?<br>

</div><div class="gmail_extra"><br clear="all"><div>--<br>Ehsan<br><<a href="http://ehsanakhgari.org/">http://ehsanakhgari.org/</a>></div>
<br><br><div class="gmail_quote">On Thu, Jul 10, 2014 at 8:52 AM, Ehsan Akhgari <span dir="ltr"><<a href="mailto:ehsan.akhgari@gmail.com" target="_blank">ehsan.akhgari@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Addressed the review comment.<br>
<div class="HOEnZb"><div class="h5"><br>
<a href="http://reviews.llvm.org/D4428" target="_blank">http://reviews.llvm.org/D4428</a><br>
<br>
Files:<br>
  lib/Driver/Tools.cpp<br>
  test/Driver/cl-link.c<br>
</div></div></blockquote></div><br></div>