<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, May 16, 2020 at 12:18 PM Greg Clayton <<a href="mailto:clayborg@gmail.com">clayborg@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="overflow-wrap: break-word;"><br><div><br><blockquote type="cite"><div>On May 15, 2020, at 7:04 PM, Eric Christopher via lldb-dev <<a href="mailto:lldb-dev@lists.llvm.org" target="_blank">lldb-dev@lists.llvm.org</a>> wrote:</div><br><div><div dir="ltr">Hi All,<div><br></div><div>We've been testing[1] a number of patches upstream by default via some pre-merge checks in phabricator. I was thinking of turning them on for lldb as well. Mostly it well just help people know whether or not they've broken lldb before they commit something, but won't stop committing or do anything else that direction.</div></div></div></blockquote><div><br></div>I am all for it! </div><div><br><blockquote type="cite"><div><div dir="ltr"><div>Let me know what you think and otherwise I'd like to turn it on in a week or so. This will also help keep the test suite a little cleaner on linux FWIW.</div></div></div></blockquote><div><br></div>Please do.</div><div><br><blockquote type="cite"><div><div dir="ltr"><div>There are a few additional links down below and if you have any questions send them my way.</div></div></div></blockquote><div><br></div>Will the lldb tests be run automagically if and only if lldb code is modified in the patch? </div></div></blockquote><div><br></div><div>I don't think our dependencies in cmake are that good for tests ... especially since lldb uses a largeish chunk of clang and llvm anyhow :)</div><div><br></div><div>-eric</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="overflow-wrap: break-word;"><div><blockquote type="cite"><div><div dir="ltr"><div>Thanks!</div><div><br></div><div>-eric</div><div><br></div><div><br>[1] <a href="https://github.com/google/llvm-premerge-checks/blob/master/docs/user_doc.md" target="_blank">https://github.com/google/llvm-premerge-checks/blob/master/docs/user_doc.md</a><br>[2] <a href="https://reviews.llvm.org/project/members/78/" target="_blank">https://reviews.llvm.org/project/members/78/</a> <br>[3] <a href="https://github.com/google/llvm-premerge-checks/issues" target="_blank">https://github.com/google/llvm-premerge-checks/issues</a><br></div></div>
_______________________________________________<br>lldb-dev mailing list<br><a href="mailto:lldb-dev@lists.llvm.org" target="_blank">lldb-dev@lists.llvm.org</a><br><a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev</a><br></div></blockquote></div><br></div></blockquote></div></div>