<div dir="ltr">I removed all these functions from the Windows DLL export list in r281747.<div><br></div><div>We don't really need weak definitions if the user provides their own, right? So, on Linux, if the user enables this coverage mode and doesn't implement one of these hooks, everything works, but nothing happens. On Windows, they'll get a link error for now.</div><div><br></div><div>That's probably going to be OK for a while. We can just tell users to define these hooks if they want to enable these coverage mdoes.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 15, 2016 at 4:17 PM, Kostya Serebryany via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: kcc<br>
Date: Thu Sep 15 18:17:02 2016<br>
New Revision: 281679<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=281679&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-<wbr>project?rev=281679&view=rev</a><br>
Log:<br>
[asan] fix window build<br>
<br>
Modified:<br>
    compiler-rt/trunk/lib/<wbr>sanitizer_common/sanitizer_<wbr>coverage_libcdep.cc<br>
<br>
Modified: compiler-rt/trunk/lib/<wbr>sanitizer_common/sanitizer_<wbr>coverage_libcdep.cc<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/sanitizer_common/sanitizer_coverage_libcdep.cc?rev=281679&r1=281678&r2=281679&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-<wbr>project/compiler-rt/trunk/lib/<wbr>sanitizer_common/sanitizer_<wbr>coverage_libcdep.cc?rev=<wbr>281679&r1=281678&r2=281679&<wbr>view=diff</a><br>
==============================<wbr>==============================<wbr>==================<br>
--- compiler-rt/trunk/lib/<wbr>sanitizer_common/sanitizer_<wbr>coverage_libcdep.cc (original)<br>
+++ compiler-rt/trunk/lib/<wbr>sanitizer_common/sanitizer_<wbr>coverage_libcdep.cc Thu Sep 15 18:17:02 2016<br>
@@ -1040,6 +1040,7 @@ uptr __sanitizer_update_counter_<wbr>bitset_a<br>
   return coverage_data.<wbr>Update8bitCounterBitsetAndClea<wbr>rCounters(bitset);<br>
 }<br>
 // Default empty implementations (weak). Users should redefine them.<br>
+#if !SANITIZER_WINDOWS  // weak does not work on Windows.<br>
 SANITIZER_INTERFACE_ATTRIBUTE SANITIZER_WEAK_ATTRIBUTE<br>
 void __sanitizer_cov_trace_cmp() {}<br>
 SANITIZER_INTERFACE_ATTRIBUTE SANITIZER_WEAK_ATTRIBUTE<br>
@@ -1064,4 +1065,5 @@ SANITIZER_INTERFACE_ATTRIBUTE SANITIZER_<br>
 void __sanitizer_cov_trace_pc_<wbr>indir() {}<br>
 SANITIZER_INTERFACE_ATTRIBUTE SANITIZER_WEAK_ATTRIBUTE<br>
 void __sanitizer_cov_trace_pc_<wbr>guard_init() {}<br>
+#endif  // !SANITIZER_WINDOWS<br>
 } // extern "C"<br>
<br>
<br>
______________________________<wbr>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br></div>