<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Wed, Nov 1, 2017 at 5:13 PM Volodymyr Sapsai <<a href="mailto:vsapsai@apple.com">vsapsai@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word;line-break:after-white-space">On Nov 1, 2017, at 16:47, Zachary Turner <<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>> wrote:<br></div><div style="word-wrap:break-word;line-break:after-white-space"><div><blockquote type="cite"><br class="m_-2225885694232795999Apple-interchange-newline"><div>This will remove the ability to use llvm-lit script even if source tree is available.<br></div></blockquote></div></div><div style="word-wrap:break-word;line-break:after-white-space"><div><div>Can you please point me to the place where llvm-lit is enabled in configure_lit_site_cfg? Asking for my education, to understand lit configuration better and to avoid breaking it. And how do you test llvm-lit script if source tree is available? I tried to check out libcxx to llvm/projects/, `ninja check-libcxx` worked fine. Though I didn’t use extra options, so this case might not reflect real-world usage.</div></div></div></blockquote><div><br></div><div>configure_lit_site_cfg doesn't actually enable llvm-lit generation, but if llvm-lit generation is already enabled, then using configure_lit_site_cfg will cause additional information to be written to the generated llvm-lit script so that you can use bin/llvm-lit <path-to-source-tree>.</div><div><br></div><div>I think all you need to do is say:</div><div><br></div><div>if (LIBCXX_STANDALONE_BUILD)</div><div>  configure_file(...</div><div>else()</div><div>   configure_lit_site_cfg(...</div><div>endif()</div><div><br></div><div>If this doesn't work for some reason though, or is too much effort, I'm not opposed to your original patch, since I think llvm-lit script generation is unconditionally disabled for libcxx right now anyway.</div><div> </div></div></div>