This is an archive of the discontinued LLVM Phabricator instance.

[clang] add tests to ExprMutAnalyzer that reproduced a crash in ASTMatchers
ClosedPublic

Authored by JonasToth on Jan 18 2019, 8:56 AM.

Details

Summary

This patch adds two unit-tests that are the result of reducing a crashing TU
when running ExprMutAnalyzer over it. They are added only to ensure the regression
that has been fixed with https://reviews.llvm.org/D56444 don't creep back.

Diff Detail

Repository
rL LLVM

Event Timeline

JonasToth created this revision.Jan 18 2019, 8:56 AM
sammccall accepted this revision.Jan 21 2019, 3:40 AM
sammccall added inline comments.
unittests/Analysis/ExprMutationAnalyzerTest.cpp
1112 ↗(On Diff #182533)

As I understand, the tests are for the same bug - can we just keep the smaller one?

This revision is now accepted and ready to land.Jan 21 2019, 3:40 AM
JonasToth updated this revision to Diff 182780.Jan 21 2019, 5:21 AM
  • use only the small reproducer
JonasToth marked an inline comment as done.Jan 21 2019, 5:23 AM
This revision was automatically updated to reflect the committed changes.