<p dir="ltr">Would be nice, if you could write a patch. I don't yet have a reduced test case (it crashes without delayed template parsing), since my creduce is still running after many hours ;)</p>
<div class="gmail_quote">On Apr 18, 2016 8:45 PM, "Michael Miller" <<a href="mailto:michael_miller@motu.com">michael_miller@motu.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">michael_miller added a comment.<br>
<br>
In <a href="http://reviews.llvm.org/D18584#403872" rel="noreferrer" target="_blank">http://reviews.llvm.org/D18584#403872</a>, @alexfh wrote:<br>
<br>
> FYI, the check has started crashing after this patch. I'll try to provide a minimal test case soon. The relevant part of the stack trace is:<br>
><br>
>   @     0x7fc9c255efa0          8  clang::Stmt::getLocStart()<br>
>   @     0x7fc9c48fdac1         64  clang::tidy::cppcoreguidelines::(anonymous namespace)::IntializerInsertion::getLocation()<br>
>   @     0x7fc9c49026d5       1696  clang::tidy::cppcoreguidelines::ProTypeMemberInitCheck::checkMissingBaseClassInitializer()<br>
>   @     0x7fc9c490424f         96  clang::tidy::cppcoreguidelines::ProTypeMemberInitCheck::check()<br>
<br>
<br>
Hmm... I can get this to crash in that location but only if I run with -fdelayed-template-parsing. I've got an easy fix and test case if that's the same issue you're running into. We should move the check at line 307 into ProTypeMemberInitCheck::check before both checks. I've got a test case I can add to cppcoreguidelines-pro-type-member-init-delayed.cpp that reproduces it. Let me know if I should write up a patch for this.<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D18584" rel="noreferrer" target="_blank">http://reviews.llvm.org/D18584</a><br>
<br>
<br>
<br>
</blockquote></div>