This is an archive of the discontinued LLVM Phabricator instance.

[lldb] Remove include/lldb/lldb-private.h
ClosedPublic

Authored by MaskRay on Aug 8 2022, 11:38 AM.

Details

Summary

The header from 62e0681afb478a4005efb6ba3598c24dc24866ee does something with
LLVM_FALLTHROUGH. Now that llvm-project has switched to C++17 and
LLVM_FALLTHROUGH uses have been migrated to [[fallthrough]], the header is
unneeded.

Diff Detail

Event Timeline

MaskRay created this revision.Aug 8 2022, 11:38 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 8 2022, 11:38 AM
Herald added a subscriber: StephenFan. · View Herald Transcript
MaskRay requested review of this revision.Aug 8 2022, 11:38 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptAug 8 2022, 11:38 AM
This revision is now accepted and ready to land.Aug 8 2022, 11:49 AM
This revision was landed with ongoing or failed builds.Aug 8 2022, 12:04 PM
This revision was automatically updated to reflect the committed changes.
clang/docs/tools/clang-formatted-files.txt