<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Tue, Nov 13, 2018 at 3:47 PM Stella Stamenova <<a href="mailto:stilis@microsoft.com">stilis@microsoft.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">





<div lang="EN-US" link="blue" vlink="purple">
<div class="m_8143167336583448376WordSection1">
<p class="MsoNormal">I am not sure if that’s the right solution for a couple of reasons:
<u></u><u></u></p>
<ol style="margin-top:0in" start="1" type="1">
<li class="m_8143167336583448376MsoListParagraph" style="margin-left:0in">As far as I can tell only clang calls use_clang (and only lld calls use_lld), while the other projects such as lld and llvm rely on the environment to be setup correctly<u></u><u></u></li><li class="m_8143167336583448376MsoListParagraph" style="margin-left:0in">Lld also has tests that invoke clang-cl and they pass – while the ones in LLDB do not, so the invocation of use_clang is not necessary for the tests to pass (maybe?)<u></u><u></u></li><li class="m_8143167336583448376MsoListParagraph" style="margin-left:0in">LLDB allows us to specify whether to use gcc or clang as well as the path and it can also have a test compiler specified via LLDB_USE_TEST_*_COMPILER, so we should first decide what
 scenarios we want to support before trying to make this work and possibly making it even more confusing and complicated<u></u><u></u></li></ol>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">Do you know what the answer for 3) is? What compilers are valid to specify for the lit/suite/unittests via the various parameters?</p></div></div></blockquote><div><br></div><div>For the unit tests, I don't think we ever specify a compiler, or we don't ever read the value.  Because a unit test shouldn't be compiling anything, it's a different kind of test.</div><div><br></div><div>For the dotest suite, specifying the compiler is important and it can definitely be gcc, but I don't think this uses the same method of going through config.cc.  In fact, I'm not sure how it determines what compiler to use at the moment, as it's been a number of years since I've looked at the dotest suite.</div><div><br></div><div>For the lit tests, I'm inclined to say we should keep things simple and only support clang for now, and add support for new compilers such as gcc if and when someone actually wants it.  Otherwise YAGNI.</div><div><br></div><div>Definitely that time will come, but it doesn't make sense to support it immediately if nobody is using it today and nobody is planning to enable it immediately.</div><div><br></div><div>So I'm tempted to say that perhaps we should just call llvm_config.use_clang() and llvm_config.use_lld() and ignore LLDB_TEST_COMPILER, which in my experience has only been a source of unnecessary complexity that never actually gets used in practice.</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div lang="EN-US" link="blue" vlink="purple"><div class="m_8143167336583448376WordSection1"><div><blockquote style="border:none;border-left:solid #cccccc 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-right:0in"><div><div><div><blockquote style="border:none;border-left:solid #cccccc 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-top:5.0pt;margin-right:0in;margin-bottom:5.0pt"><p class="MsoNormal" style="margin-bottom:12.0pt">
<u></u><u></u></p>
</blockquote>
</div>
</div>
</div>
</blockquote>
</div>
</div></div></blockquote></div></div>