This is an archive of the discontinued LLVM Phabricator instance.

[include-fixer] Sort headers after inserting new headers.
ClosedPublic

Authored by ioeric on May 18 2016, 8:01 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

ioeric updated this revision to Diff 57624.May 18 2016, 8:01 AM
ioeric retitled this revision from to [include-fixer] Sort headers after inserting new headers..
ioeric updated this object.
ioeric updated this revision to Diff 57626.May 18 2016, 8:11 AM
  • Added compile dependency and fixed nits.
ioeric added subscribers: cfe-commits, hokein, djasper, klimek.
ioeric updated this revision to Diff 57745.May 19 2016, 1:15 AM
  • Passed Headers into IncludeFixerActionFactory so that we can know which headers are added.
bkramer accepted this revision.May 19 2016, 1:19 AM
bkramer edited edge metadata.

lgtm

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