This is an archive of the discontinued LLVM Phabricator instance.

[lld] Replace Symbolize.h with DIContext.h in lld's COFF lib
ClosedPublic

Authored by noajshu on Dec 13 2021, 12:05 PM.

Details

Summary

lld only needs DIContext.h which it gets through Symbolize.h -> SymbolizableModule.h -> DIContext.h. This replaces it with a direct include of DIContext.h to avoid any confusion and pulling in unnecessary headers.

Diff Detail

Event Timeline

phosek accepted this revision.Dec 13 2021, 1:21 PM

LGTM

This revision is now accepted and ready to land.Dec 13 2021, 1:21 PM
Herald added a project: Restricted Project. · View Herald TranscriptDec 13 2021, 2:17 PM