This is an archive of the discontinued LLVM Phabricator instance.

<<Replace this line with your revision title> [analyzer][CrossTU][NFC] Fix sanitizer test failure
AbandonedPublic

Authored by r.stahl on Apr 23 2019, 4:59 AM.

Details

Reviewers
xazax.hun
Summary

Missing break/fallthrough

Diff Detail

Event Timeline

r.stahl created this revision.Apr 23 2019, 4:59 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 23 2019, 4:59 AM
xazax.hun accepted this revision.Apr 23 2019, 5:04 AM

Feel free to commit such trivial fixes without reviews. Alternatively, you could use LLVM_FALLTHROUGH, but I have no strong preference in this case.

This revision is now accepted and ready to land.Apr 23 2019, 5:04 AM

@xazax.hun Yes, I planned to just commit. Set you as Subscriber not Reviewer in Arc. Was just a bit confused why it fails at first :)