This is an archive of the discontinued LLVM Phabricator instance.

[Frontend] Remove ShowIncludesPretendHeader
ClosedPublic

Authored by smeenai on Jun 16 2023, 3:47 PM.

Details

Summary

It hasn't been written to since https://reviews.llvm.org/D46652, so it
was always empty. I don't have enough context on that diff to know if
the removal of the write to ShowIncludesPretendHeader in that diff was
intentional, but no one's complained about it for five years, so I
assume we're okay to just get rid of it entirely.

Diff Detail

Event Timeline

smeenai created this revision.Jun 16 2023, 3:47 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 16 2023, 3:47 PM
smeenai requested review of this revision.Jun 16 2023, 3:47 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 16 2023, 3:47 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
hans accepted this revision.Jun 20 2023, 3:19 AM

lgtm

This revision is now accepted and ready to land.Jun 20 2023, 3:19 AM
This revision was automatically updated to reflect the committed changes.