<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Oct 11, 2013 at 2:48 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="im">> Since you've got it setup, have you run this against the GDB 7.5 test suite?<br>

<br>
</div>Ah, thanks. I had checked just the case that broke before. It is a<br>
alias from derived to base. We don't produce such an alias anymore<br>
since it would also hit the "same comdat rule". I will add an explicit<br>
test of that.<br>
<br>
Running the gdb tests now I got:<br>
<br>
# of expected passes 4737<br></blockquote><div><br></div><div>If you're running make check -j/FORCE_PARALLEL, the print out at the end won't be the proper summary (it'll just be whichever shard finished last). If you check in testsuite/gdb.sum you should have a summary at the end with somewhere around 21000 tests. ( see for example the end of <a href="http://lab.llvm.org:8011/builders/clang-x86_64-ubuntu-gdb-75/builds/9540/steps/gdb-75-check/logs/dg.sum">http://lab.llvm.org:8011/builders/clang-x86_64-ubuntu-gdb-75/builds/9540/steps/gdb-75-check/logs/dg.sum</a> )</div>
<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
# of unexpected successes 1<br>
# of expected failures 23<br>
# of unknown successes 2<br>
# of known failures 7<br>
# of untested testcases 6<br>
# of unsupported tests 4<br>
<br>
both with and without the patch.<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div></div>