This is an archive of the discontinued LLVM Phabricator instance.

[NFC] Header cleanup
ClosedPublic

Authored by kevgs on May 10 2016, 6:17 AM.

Details

Summary

Removed unused headers, replaced some headers with forward class declarations

Diff Detail

Repository
rL LLVM

Event Timeline

kevgs updated this revision to Diff 56703.May 10 2016, 6:17 AM
kevgs retitled this revision from to [NFC] Header cleanup.
kevgs updated this object.
kevgs added a reviewer: mehdi_amini.
kevgs added a subscriber: cfe-commits.

Was this change created by a tool?

kevgs added a comment.May 10 2016, 8:47 AM

Sadly, I dont know such a tool. I was using a simple script to find unused headers ack --cpp -l 'SmallString\.h"' | xargs grep -L 'SmallString[ (<I]' | xargs grep -n --color=auto 'SmallString'

mehdi_amini accepted this revision.Jul 12 2016, 4:14 PM
mehdi_amini edited edge metadata.

Looks like a nice cleanup.

This revision is now accepted and ready to land.Jul 12 2016, 4:14 PM
kevgs added a comment.Jul 16 2016, 1:02 AM

Good. I have no commit rights.

Can you rebase? I can't apply it right now.

kevgs updated this revision to Diff 64303.Jul 18 2016, 5:01 AM
kevgs edited edge metadata.

Updated to current trunk.

This revision was automatically updated to reflect the committed changes.