<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, 15 Jan 2022, 4:30 am Stella Stamenova via Phabricator, <<a href="mailto:reviews@reviews.llvm.org">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">stella.stamenova added a comment.<br>
<br>
In D117363#3245124 <<a href="https://reviews.llvm.org/D117363#3245124" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D117363#3245124</a>>, @gkistanova wrote:<br>
<br>
>> I think they should be set in just one place to make the code more readable.<br>
><br>
> I was about to suggest the same.<br>
><br>
> It might be simpler and more straight forward to revert D116972 <<a href="https://reviews.llvm.org/D116972" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D116972</a>> and just explicitly specify the needed `LLVM_LIT_ARGS` in the `extra_cmake_args`.<br>
> If you would like to set `-v` and `-j` by default, that could be done by something like this:<br>
><br>
>   defult_lit_args = '-v'<br>
>   if if jobs:<br>
>       defult_lit_args += " -j{jobs}".format(jobs=jobs)<br>
>   CmakeCommand.applyDefaultOptions(cmake_options, [<br>
>           ('-DLLVM_LIT_ARGS=', defult_lit_args),<br>
>           ])<br>
<br>
What about adding a separate parameter for the jobs for the tests? That way a different level of parallelism could be specified for build and test? That's the route I would have gone.<br></blockquote></div></div><div dir="auto">This sounds like a good idea. I have to make a couple of changes for Arm Windows bot. Let me give this a bit more thought before I make a final decision on this.</div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D117363/new/" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D117363/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D117363" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D117363</a><br>
<br>
</blockquote></div></div></div>