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 ;) On Apr 18, 2016 8:45 PM, "Michael Miller" <michael_mil...@motu.com> wrote:
> michael_miller added a comment. > > In http://reviews.llvm.org/D18584#403872, @alexfh wrote: > > > 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: > > > > @ 0x7fc9c255efa0 8 clang::Stmt::getLocStart() > > @ 0x7fc9c48fdac1 64 > clang::tidy::cppcoreguidelines::(anonymous > namespace)::IntializerInsertion::getLocation() > > @ 0x7fc9c49026d5 1696 > clang::tidy::cppcoreguidelines::ProTypeMemberInitCheck::checkMissingBaseClassInitializer() > > @ 0x7fc9c490424f 96 > clang::tidy::cppcoreguidelines::ProTypeMemberInitCheck::check() > > > 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. > > > Repository: > rL LLVM > > http://reviews.llvm.org/D18584 > > > >
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits