<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Apr 14, 2017 at 9:53 AM, Reid Kleckner via Phabricator 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"><span class="">rnk added inline comments.<br>
<br>
<br>
================<br>
Comment at: llvm/trunk/include/llvm/<wbr>ProfileData/InstrProfData.inc:<wbr>666<br>
+<br>
+#ifdef _WIN32<br>
 /* Runtime section names and name strings.  */<br>
----------------<br>
</span><span class="">rnk wrote:<br>
> This ifdef breaks cross compiling. In particular, llvm/test/Instrumentation/<wbr>InstrProfiling/X86/alloc.ll is failing on Windows since this change. LLVM should choose the section name at runtime based on the target triple, not the host defines.<br>
</span>Oops, I think I spoke too soon. I don't think I understand why the test is failing yet.<br>
<span class="im HOEnZb"><br></span></blockquote><div><br></div><div>To clarify -- the ifdef is not used by compiler but only runtime, so it is not the cause of the failure. That patch already enables compiler to check target type and decide what name scheme to use. However, in target object lowering code, the object format is not passed in thus the naming for the host is returned -- it is a just a bug in the patch.</div><div><br></div><div>David</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="im HOEnZb">
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D32023" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D32023</a><br>
<br>
<br>
<br>
</span><div class="HOEnZb"><div class="h5">______________________________<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>
</div></div></blockquote></div><br></div></div>