That’s fine <br><div class="gmail_quote"><div dir="ltr">On Tue, Dec 12, 2017 at 8:13 AM Don Hinton <<a href="mailto:hintonda@gmail.com">hintonda@gmail.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">Sure, but I'll need to do it in order:<div><br></div><div>  D41055</div><div>  D40971 (this patch)</div><div>  D40972</div><div><br></div><div>Just wanted to make sure that's okay before starting the process.</div><div><br></div><div>thanks...</div></div><div dir="ltr"><div>don</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Dec 12, 2017 at 8:05 AM, 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">Lgtm do you want to try committing today?<div class="m_-9015661098876290719HOEnZb"><div class="m_-9015661098876290719h5"><br><div class="gmail_quote"><div dir="ltr">On Sat, Dec 9, 2017 at 9:36 PM Don Hinton 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">hintonda added inline comments.<br>
<br>
<br>
================<br>
Comment at: <a href="http://lit.site.cfg.py.in:25" rel="noreferrer" target="_blank">lit.site.cfg.py.in:25</a><br>
+# Let the main config do the real work.<br>
+lit_config.load_config(config, "@DEBUGINFO_TESTS_SOURCE_DIR@/<a href="http://lit.config.py" rel="noreferrer" target="_blank">lit.config.py</a>")<br>
----------------<br>
zturner wrote:<br>
> hintonda wrote:<br>
> > This is how lit find <a href="http://lit.config.py" rel="noreferrer" target="_blank">lit.config.py</a>.<br>
> ><br>
> > I'll test the monorepo variant and get back to you.<br>
> Yes, but you need to be able to type `llvm-lit.py path/to/debuginfo-tests`, and if you do that, lit is going to be looking specifically for a `<a href="http://lit.cfg.py" rel="noreferrer" target="_blank">lit.cfg.py</a>` and won't find it<br>
Changing the name back to <a href="http://lit.cfg.py" rel="noreferrer" target="_blank">lit.cfg.py</a> was required for the monorepo, but also required a change to clang, D41055, which needs to land first -- hope I set the dependency correctly.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D40971" rel="noreferrer" target="_blank">https://reviews.llvm.org/D40971</a><br>
<br>
<br>
<br>
</blockquote></div>
</div></div></blockquote></div><br></div>
</blockquote></div>