<div dir="ltr">I thought about something like that too.  Let me try to get all these other patches landed first then I'll think about what to do.</div><br><div class="gmail_quote"><div dir="ltr">On Mon, Mar 20, 2017 at 9:25 AM 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"><div dir="ltr" class="gmail_msg">Aha, I see that now. What do you think about creating a test-only module that both unit tests can depend on? The llvm folder-globbing means we would have to put it in a subfolder (unittests/Utility/lib ?), which makes is somewhat annoying, but I still think that's better than having the test code in the main module.</div><div class="gmail_extra gmail_msg"><br class="gmail_msg"><div class="gmail_quote gmail_msg">On 20 March 2017 at 13:04, Zachary Turner <span dir="ltr" class="gmail_msg"><<a href="mailto:zturner@google.com" class="gmail_msg" target="_blank">zturner@google.com</a>></span> wrote:<br class="gmail_msg"><blockquote class="gmail_quote gmail_msg" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The reason was because two different tests from two different unittest executables need it, but we have no good way to share code across unittests like this.  Open to suggestions though<div class="m_-7471592986794405978HOEnZb gmail_msg"><div class="m_-7471592986794405978h5 gmail_msg"><br class="gmail_msg"><div class="gmail_quote gmail_msg"><div dir="ltr" class="gmail_msg">On Mon, Mar 20, 2017 at 4:02 AM Pavel Labath via Phabricator <<a href="mailto:reviews@reviews.llvm.org" class="gmail_msg" target="_blank">reviews@reviews.llvm.org</a>> wrote:<br class="gmail_msg"></div><blockquote class="gmail_quote gmail_msg" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">labath added inline comments.<br class="m_-7471592986794405978m_-5660249965042317437gmail_msg gmail_msg">
<br class="m_-7471592986794405978m_-5660249965042317437gmail_msg gmail_msg">
<br class="m_-7471592986794405978m_-5660249965042317437gmail_msg gmail_msg">
================<br class="m_-7471592986794405978m_-5660249965042317437gmail_msg gmail_msg">
Comment at: lldb/include/lldb/Utility/TildeExpressionResolver.h:62<br class="m_-7471592986794405978m_-5660249965042317437gmail_msg gmail_msg">
+<br class="m_-7471592986794405978m_-5660249965042317437gmail_msg gmail_msg">
+class MockTildeExpressionResolver : public TildeExpressionResolver {<br class="m_-7471592986794405978m_-5660249965042317437gmail_msg gmail_msg">
+  llvm::StringRef CurrentUser;<br class="m_-7471592986794405978m_-5660249965042317437gmail_msg gmail_msg">
----------------<br class="m_-7471592986794405978m_-5660249965042317437gmail_msg gmail_msg">
I find it very strange to see the Mock object in a regular header file (and I also don't see a reason why we would need that). Could we get rid of this? (perhaps by declaring it in a test-only header file if necessary)<br class="m_-7471592986794405978m_-5660249965042317437gmail_msg gmail_msg">
<br class="m_-7471592986794405978m_-5660249965042317437gmail_msg gmail_msg">
<br class="m_-7471592986794405978m_-5660249965042317437gmail_msg gmail_msg">
<a href="https://reviews.llvm.org/D31129" rel="noreferrer" class="m_-7471592986794405978m_-5660249965042317437gmail_msg gmail_msg" target="_blank">https://reviews.llvm.org/D31129</a><br class="m_-7471592986794405978m_-5660249965042317437gmail_msg gmail_msg">
<br class="m_-7471592986794405978m_-5660249965042317437gmail_msg gmail_msg">
<br class="m_-7471592986794405978m_-5660249965042317437gmail_msg gmail_msg">
<br class="m_-7471592986794405978m_-5660249965042317437gmail_msg gmail_msg">
</blockquote></div>
</div></div></blockquote></div><br class="gmail_msg"></div>
</blockquote></div>