This is an archive of the discontinued LLVM Phabricator instance.

[clangd] Avoid including HeuristicResolver.h from ParsedAST.h
ClosedPublic

Authored by nridge on Apr 25 2021, 5:04 PM.

Diff Detail

Event Timeline

nridge created this revision.Apr 25 2021, 5:04 PM
nridge requested review of this revision.Apr 25 2021, 5:04 PM
nridge added a comment.EditedApr 25 2021, 5:05 PM

Motivation: ParsedAST.h is widely included (within clangd), and now that ParsedAST stores HeuristicResolver by unique_ptr, the header dependency is no longer necessary.

sammccall accepted this revision.Apr 28 2021, 3:17 AM
This revision is now accepted and ready to land.Apr 28 2021, 3:17 AM
This revision was landed with ongoing or failed builds.May 2 2021, 9:55 PM
This revision was automatically updated to reflect the committed changes.