<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Feb 20, 2018 at 10:20 PM, Frederic Riss via lldb-commits <span dir="ltr"><<a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">Author: friss<br>
Date: Tue Feb 20 22:20:03 2018<br>
New Revision: 325666<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=325666&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-pr<wbr>oject?rev=325666&view=rev</a><br>
Log:<br>
Fix TestAppleTypesIsProduced after r324226<br>
<br>
This test was accessing self.debug_info, which doesn't exist anymore. For<br>
some reason the macOS bots are skipping this test because they think the<br>
compiler is not clang. We'll look into this separately.<br></blockquote><div><br></div><div>I stumbled upon something similar this morning so I decided to take a look.</div><div>The reason why this fails is that the test run is conditional to:</div><div><br></div><div>        if not self.getCompiler().endswith('<wbr>clang'):</div><div><br></div><div>If you run with an in-tree clang, you might end up with getCompiler() returning:</div><div><br></div><div>/Users/davide/work/llvm-<wbr>monorepo/build/bin/clang-7.0 <br></div><div><br></div><div>which fails the check.</div><div><br></div><div>The reason why you're seeing this only locally is maybe because you're running with the clang provided by the system instead of the one provided in-tree.</div><div>The mismatch has been a cause of headache for me in the past already.</div><div><br></div><div>Regardless, the way this test was setup was good to understand this mistake (so we should probably audit all the tests looking for this pattern), but I have my concerns about the place where the test is located.</div><div>The program is just making sure clang emits the correct sections, so we might consider moving this test to clang.</div><div><br></div><div>Thanks,</div><div><br></div><div>--</div><div>Davide</div></div></div></div>