<div dir="ltr">I've just sent <a href="https://reviews.llvm.org/D65609">https://reviews.llvm.org/D65609</a> for review.<br><div><br></div><div>It obviously needs to figure out how to tell if gdb is present. I'm not sure how to communicate from find_program inside cmake all the way down into UNSUPPORTED in the test file, but that is a minor detail.</div><div><br></div><div>There are a lot of judgement calls on how things look. We tried to match libstdc++ where possible.</div><div><br></div><div>Also, happy to change directories or whatever.</div><div><br></div><div>We also need to figure out how to ship it to users. libstdc++.so uses a .debug_gdb_scripts section to point to an absolute path where it is installed. Static linking is trickier.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jul 22, 2019 at 11:23 AM Louis Dionne via libcxx-dev <<a href="mailto:libcxx-dev@lists.llvm.org">libcxx-dev@lists.llvm.org</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 Jul 19, 2019, at 16:39, Duncan Exon Smith via libcxx-dev <<a href="mailto:libcxx-dev@lists.llvm.org" target="_blank">libcxx-dev@lists.llvm.org</a>> wrote:</div><br class="gmail-m_4336258515790908388Apple-interchange-newline"><div><div dir="auto"><div dir="ltr">You should be able to sniff in lit.local.config whether (a new enough) gdb is installed and mark the tests unsupported if not.  Unsupported is better than xfail here. </div></div></div></blockquote><div><br></div><div>This assumes that the tests are even using `lit`. I don’t know whether that’s the case. Let’s wait for the PR to be up!</div><div><br></div><div>Louis</div><br><blockquote type="cite"><div><div dir="auto"><div dir="ltr"><br><blockquote type="cite">On Jul 19, 2019, at 20:09, Louis Dionne via libcxx-dev <<a href="mailto:libcxx-dev@lists.llvm.org" target="_blank">libcxx-dev@lists.llvm.org</a>> wrote:<br><br></blockquote></div><blockquote type="cite"><div dir="ltr"><br><div><blockquote type="cite"><div>On Jul 19, 2019, at 13:38, Sterling Augustine <<a href="mailto:saugustine@google.com" target="_blank">saugustine@google.com</a>> wrote:</div><br class="gmail-m_4336258515790908388Apple-interchange-newline"><div><div dir="ltr">I'm working on this today. Question:<div><br></div><div>What directories would be best for the python package and the associated tests? They don't seem to fit well in any current directories.</div></div></div></blockquote><div><br></div><div>Under libcxx/test, we could add a new directory like libcxx/test/gdb. We need to keep this out of the directories normally considered by lit, but I think that’s the only hard requirement.</div><br><blockquote type="cite"><div><div dir="ltr"><div><br></div><div>If gdb isn't installed on the machine, should they fail or not run? There are tradeoffs with both approaches.</div></div></div></blockquote><div><br></div><div>They should not run unless there’s a compelling reason to do otherwise. We don’t want to tie passing the libc++ test suite to having gdb installed!</div><div><br></div><div>Note that whatever approach we use, it would be nice if we could use the same approach for the lldb formatters once/if the LLDB folks decide to send them our way.</div><div><br></div><div>Louis</div><br><blockquote type="cite"><div><div dir="ltr"><div><br></div><div>Getting them shipped to actual users is a different can of worms, but we can look at that when we actually have something to work with.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jul 18, 2019 at 9:23 AM Sterling Augustine <<a href="mailto:saugustine@google.com" target="_blank">saugustine@google.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 dir="ltr">Yes they will need gdb to run, and we will go through the full review process with all of it.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jul 18, 2019 at 9:08 AM Louis Dionne <<a href="mailto:ldionne@apple.com" target="_blank">ldionne@apple.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>I'm assuming the tests need GDB in order to run -- is that correct? I assume you'll put up a Phab review when you're ready?<div><br></div><div>Louis<br><div><br><blockquote type="cite"><div>On Jul 17, 2019, at 16:40, Sterling Augustine <<a href="mailto:saugustine@google.com" target="_blank">saugustine@google.com</a>> wrote:</div><br class="gmail-m_4336258515790908388gmail-m_7382432424048184403gmail-m_-5214308898581693393Apple-interchange-newline"><div><div dir="ltr"><div dir="ltr">On Wed, Jul 17, 2019 at 12:53 PM Louis Dionne <<a href="mailto:ldionne@apple.com" target="_blank">ldionne@apple.com</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div>What do the tests look like? Is that something that we can reasonably run as part of the libc++ test suite?</div></blockquote><div><br></div><div>Yes. I would upstream the tests them as part of upstreaming the printers. I'm working on this.</div></div></div>
</div></blockquote></div><br></div></div></blockquote></div>
</blockquote></div>
</div></blockquote></div><br><span>_______________________________________________</span><br><span>libcxx-dev mailing list</span><br><span><a href="mailto:libcxx-dev@lists.llvm.org" target="_blank">libcxx-dev@lists.llvm.org</a></span><br><span><a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/libcxx-dev" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/libcxx-dev</a></span><br></div></blockquote></div>_______________________________________________<br>libcxx-dev mailing list<br><a href="mailto:libcxx-dev@lists.llvm.org" target="_blank">libcxx-dev@lists.llvm.org</a><br><a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/libcxx-dev" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/libcxx-dev</a><br></div></blockquote></div><br></div>_______________________________________________<br>
libcxx-dev mailing list<br>
<a href="mailto:libcxx-dev@lists.llvm.org" target="_blank">libcxx-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/libcxx-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/libcxx-dev</a><br>
</blockquote></div>