<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">Hi George,<br></div><div class="gmail_quote"><br>On Mon, Apr 2, 2018 at 5:02 PM, George Karpenkov <span dir="ltr"><<a href="mailto:ekarpenkov@apple.com" target="_blank">ekarpenkov@apple.com</a>></span> wrote:<br><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">Seems useful, but the current output is quite verbose.<div>Wouldn’t just</div><div><br></div><div>RUN:1; ls</div><div><br></div><div>be sufficient in place of</div><div><br></div><div>'RUN: at line 1’; ls</div><div><br></div><div>?</div></div></blockquote><div><br></div><div>My only argument against your shorter version is that its meaning might not be obvious to someone seeing it for the first time.  Nevertheless, it would certainly suffice for me.<br></div><div> </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"><div><br></div><div>I’m also curious about the implementation, I recall there was a discussion about forcing all tests to use internal shell?</div></div></blockquote><div><br>Do you have a link?<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"><div>In that case a trick with a no-op “:” command would not be necessary.</div></div></blockquote><br></div><div class="gmail_quote">Agreed.  Without having seen the discussion, my guess it that usage of the external shell isn't going away immediately, and this patch seems immediately useful.  It's also fairly small, so it should be easy to migrate to an implementation tailored to the internal shell if usage of the external shell goes away.<br></div><div class="gmail_quote"><div><br></div><div>Thanks.<br><br></div><div>Joel <br></div><div><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"><div><br></div><div>George</div><div><div><br><blockquote type="cite"><div><div class="h5"><div>On Apr 2, 2018, at 1:25 PM, Joel E. Denny via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:</div><br class="m_-9142879389048881915Apple-interchange-newline"></div></div><div><div><div class="h5"><div dir="ltr"><div class="m_-9142879389048881915gmail-phui-property-list-text-content"><div class="m_-9142879389048881915gmail-phabricator-remarkup"><p>When some lit tests fail, I find it time-consuming to locate the source line of the failing RUN command.  Running lit with -vv (or just -v in the case of the internal shell) makes it easier because it prints an expanded version of each RUN command as it executes, but it can still be challenging because lit doesn't provide the RUN command's source line number.</p><p>While I haven't personally seen failures in the following tests, grep helped me discover them as extreme examples where finding the RUN source line should prove challenging:</p><p>1. clang's test/Driver/linux-ld.c has 892 total RUN lines</p><p>2. clang's test/Driver/arm-cortex-cpus.c has 424 RUN lines after concatenation for line continuations</p><p>To address this issue, I wrote the following patch to introduce RUN source line numbers into lit's -vv output (or -v for the internal shell), but the patch needs a review:<br></p><p><a href="https://reviews.llvm.org/D44598" target="_blank">https://reviews.llvm.org/<wbr>D44598</a></p>I've been using this patch for a couple of weeks, and it's a big improvement for me.  If there's some existing feature that people already use to handle this issue, please let me know so I can improve my usage of lit.<br><br></div><div class="m_-9142879389048881915gmail-phabricator-remarkup">Thanks.<br></div><div class="m_-9142879389048881915gmail-phabricator-remarkup"><br></div><div class="m_-9142879389048881915gmail-phabricator-remarkup">Joel</div></div><br></div></div></div>
______________________________<wbr>_________________<br>LLVM Developers mailing list<br><a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br><a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-dev</a><br></div></blockquote></div><br></div></div></blockquote></div><br></div></div>