This is an archive of the discontinued LLVM Phabricator instance.

[include-fixer] Ignore non-scoped enum declaration during search.
ClosedPublic

Authored by hokein on May 18 2016, 1:20 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

hokein updated this revision to Diff 57572.May 18 2016, 1:20 AM
hokein retitled this revision from to [include-fixer] Ignore non-scoped enum declaration during search..
hokein updated this object.
hokein set the repository for this revision to rL LLVM.
hokein added subscribers: ioeric, cfe-commits.
bkramer accepted this revision.May 18 2016, 1:49 AM
bkramer edited edge metadata.

lg

include-fixer/SymbolIndexManager.cpp
45 ↗(On Diff #57572)

It looks like we should rewrite this loop with early exits eventually. Not sure if that's a net win right now so I'm also fine with doing that when the code becomes more complex the next time.

This revision is now accepted and ready to land.May 18 2016, 1:49 AM
This revision was automatically updated to reflect the committed changes.