<div dir="ltr">Ahh, sorry.  While I feel strongly about 1 long term, going from 4 -> 2 is already a win, so I see no reason to block this over that.  We can then work on going from 2 -> 1 later.</div><br><div class="gmail_quote"><div dir="ltr">On Thu, Oct 26, 2017 at 4:57 PM Pavel Labath <<a href="mailto:labath@google.com">labath@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I haven't put this in yet. I was still waiting for a reaction from<br>
Zachary about the two cmake vars, as he felt pretty strongly about<br>
unifying to one. (I didn't want to do that as I would have to write<br>
code for transforming /usr/bin/arm-linux-gnu-gcc-4.7 (and likes) into<br>
/usr/bin/arm-linux-gnu-g++-4.7, which would probably end up very<br>
ugly).<br>
<br>
On 26 October 2017 at 16:50, Zachary Turner <<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>> wrote:<br>
> I think you now need to use `-DLLDB_TEST_C_COMPILER` and<br>
> `-DLLDB_TEST_CXX_COMPILER`<br>
><br>
> On Thu, Oct 26, 2017 at 4:40 PM Paul Robinson via Phabricator<br>
> <<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>> wrote:<br>
>><br>
>> probinson added a comment.<br>
>><br>
>> Has this gone in?  I'm wondering because I starting playing with the<br>
>> monorepo, ran cmake with -DLLDB_TEST_COMPILER=$PWD/bin/clang, and today's<br>
>> test failure seems to be trying to build the test program with the system<br>
>> compiler (gcc) rather than my copy of clang.  But it looks like you're<br>
>> deprecating LLDB_TEST_COMPILER?<br>
>><br>
>><br>
>> <a href="https://reviews.llvm.org/D39215" rel="noreferrer" target="_blank">https://reviews.llvm.org/D39215</a><br>
>><br>
>><br>
>><br>
><br>
</blockquote></div>