This is an archive of the discontinued LLVM Phabricator instance.

[clang][Sema] Fix a copy/paste bug in ~Sema()
ClosedPublic

Authored by tbaeder on Aug 20 2023, 4:59 AM.

Diff Detail

Event Timeline

tbaeder created this revision.Aug 20 2023, 4:59 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 20 2023, 4:59 AM
tbaeder requested review of this revision.Aug 20 2023, 4:59 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 20 2023, 4:59 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
aaron.ballman accepted this revision.Aug 21 2023, 5:36 AM
aaron.ballman added a subscriber: aaron.ballman.

LGTM! I have absolutely no idea how you would test the Sema destructor, so I think it's fine that this comes with no test coverage. Good catch!

This revision is now accepted and ready to land.Aug 21 2023, 5:36 AM
This revision was landed with ongoing or failed builds.Sep 4 2023, 6:33 AM
This revision was automatically updated to reflect the committed changes.