<div dir="ltr">We don't support running the test suite on Windows with MSVC.  We run it with clang targeting windows instead.  So anyone running the test suite on Windows is already using clang, and we can just specify a linux triple to get an ELF binary.</div><br><div class="gmail_quote"><div dir="ltr">On Mon, Apr 24, 2017 at 2:47 PM Eugene Zemtsov <<a href="mailto:ezemtsov@google.com">ezemtsov@google.com</a>> wrote:<br></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_default" style="font-family:verdana,sans-serif">Microsoft Visual C++ first comes to my mind.</div></div><div class="gmail_extra"></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Apr 24, 2017 at 2:31 PM, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@google.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">Which C compilers do we have that are not capable of producing ELF binaries?   GCC on Windows maybe?  That's not even a supported test configuration.</div><div class="m_6045879086131298686HOEnZb"><div class="m_6045879086131298686h5"><br><div class="gmail_quote"><div dir="ltr">On Mon, Apr 24, 2017 at 2:29 PM Pavel Labath via Phabricator <<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">labath added a comment.<br>
<br>
In <a href="https://reviews.llvm.org/D32434#735877" rel="noreferrer" target="_blank">https://reviews.llvm.org/D32434#735877</a>, @eugene wrote:<br>
<br>
> Is it  really necessary to check in binary ELF files?<br>
>  I understand that we don't always have a C compiler capable of producing ELF files, but maybe it's ok to skip this test on those platforms.<br>
<br>
<br>
That is something very I am very much trying to avoid, as that means people on those platforms cannot validate their changes. (And I'm not sure if I even want to be running a compiler for a test at this level, as that introduces a level of nondeterminism.)<br>
<br>
That said, I do agree we should be carefully about adding lots of binary bloat. I have been trying to put this off until we have more of these, but maybe I could try integrating this with obj2yaml -- it does not seem to preserve program headers, but I don't actually need those for this test.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D32434" rel="noreferrer" target="_blank">https://reviews.llvm.org/D32434</a><br>
<br>
<br>
<br>
</blockquote></div>
</div></div></blockquote></div><br><br clear="all"><div><br></div></div><div class="gmail_extra">-- <br><div class="m_6045879086131298686gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><font face="verdana, sans-serif">Thanks,</font><div><font face="verdana, sans-serif">Eugene Zemtsov.</font></div></div></div>
</div></blockquote></div>