This is an archive of the discontinued LLVM Phabricator instance.

Ignore macro expansions when scanning for fallthrough comments
AbandonedPublic

Authored by tbaeder on Mar 2 2020, 8:10 AM.

Details

Reviewers
tstellar
Summary

When scanning for fallthrough comments, it's not useful to go to the definition of a macro. We instead only want to search at the expanded source code location. Includes a test that failed before.

Diff Detail

Event Timeline

tbaeder created this revision.Mar 2 2020, 8:10 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 2 2020, 8:10 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
tbaeder abandoned this revision.Mar 3 2020, 11:02 PM

fallthrough comment detection has been reverted, so abandon this in the meantime.