<div dir="auto"><div>I'll take a look at the regression, thanks.</div><div dir="auto">As noted in the other review thread, the new unit test you linked is broken: the code snippet doesn't compile (a&&), which seems to be the cause of the crash, or at least masks it.<br><br><div class="gmail_quote" dir="auto"><div dir="ltr">On Tue, Jan 8, 2019, 23:20 Jonas Toth via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">JonasToth added a comment.<br>
<br>
I still see the unit-test crashing for `ExprMutAnalyzer` (just apply these tests <a href="https://github.com/JonasToth/clang/blob/fix_crash/unittests/Analysis/ExprMutationAnalyzerTest.cpp" rel="noreferrer noreferrer" target="_blank">https://github.com/JonasToth/clang/blob/fix_crash/unittests/Analysis/ExprMutationAnalyzerTest.cpp</a> and run `check-clang-unit`)<br>
The clang-tidy check does not crash anymore, but `readability-function-size` regresses. It probably doesn't properly count for lambdas anymore.<br>
<br>
<br>
Repository:<br>
  rC Clang<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D56444/new/" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D56444/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D56444" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D56444</a><br>
<br>
<br>
<br>
</blockquote></div></div></div>