This is an archive of the discontinued LLVM Phabricator instance.

[lldb][CPlusPlus][NFCI] Remove redundant construction of ClangASTImporter
ClosedPublic

Authored by Michael137 on Aug 15 2023, 9:07 AM.

Details

Summary

The usage of this variable was removed in 4f14c17df70916913d71914343dd4f6c709e218d.

This is no longer used inside this file. Since the call to
GetPersistentExpressionStateForLanguage has side-effects I marked this
NFCI. But there is no good reason we need this here.

Diff Detail

Event Timeline

Michael137 created this revision.Aug 15 2023, 9:07 AM
Herald added a project: Restricted Project. · View Herald Transcript
Herald added a subscriber: martong. · View Herald Transcript
Michael137 requested review of this revision.Aug 15 2023, 9:07 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 15 2023, 9:07 AM
aprantl accepted this revision.Aug 15 2023, 12:45 PM
This revision is now accepted and ready to land.Aug 15 2023, 12:45 PM