This is an archive of the discontinued LLVM Phabricator instance.

[clang-tidy] Fix a crash in fuchsia-multiple-inheritance
ClosedPublic

Authored by ilya-biryukov on Jul 26 2018, 9:15 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

ilya-biryukov created this revision.Jul 26 2018, 9:15 AM
hokein accepted this revision.Jul 26 2018, 10:42 PM

LGTM.

test/clang-tidy/fuchsia-multiple-inheritance-crash.cpp
1 ↗(On Diff #157503)

nit: instead of create a new file, we can put this test case in the existing test/clang-tidy/fuchsia-multiple-inheritance.cpp.

This revision is now accepted and ready to land.Jul 26 2018, 10:42 PM
ilya-biryukov marked an inline comment as done.
  • Move the test into an existing file
This revision was automatically updated to reflect the committed changes.

Sorry for sending this in instead of waiting for D49158 to land, I hadn't noticed it earlier