This is an archive of the discontinued LLVM Phabricator instance.

[clang-tidy] Fix crash in --dump-config
ClosedPublic

Authored by njames93 on Apr 15 2023, 11:11 AM.

Details

Summary

Fixes a crash in dump-config when checks emit warnings trying to read the config

Fixes https://github.com/llvm/llvm-project/issues/50876

Co-authored-by: Piotr Zegar <me@piotrzegar.pl>

Diff Detail

Event Timeline

njames93 created this revision.Apr 15 2023, 11:11 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 15 2023, 11:11 AM
Herald added a subscriber: xazax.hun. · View Herald Transcript
njames93 requested review of this revision.Apr 15 2023, 11:11 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 15 2023, 11:11 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript

@PiotrZSL I hope you don't mind, I've put just the crash fix up here for D146842 and marked you as the coauthor.

PiotrZSL accepted this revision.Apr 15 2023, 11:40 AM
This revision is now accepted and ready to land.Apr 15 2023, 11:40 AM
This revision was automatically updated to reflect the committed changes.