This is an archive of the discontinued LLVM Phabricator instance.

Remove isIgnored()-test that is more expensive than the analysis behind it
ClosedPublic

Authored by djasper on Jan 1 2017, 7:16 AM.

Details

Reviewers
rsmith
Summary

In many translation units I have tried, the calls to isIgnored() removed in this patch are more expensive than doing the analysis that is behind it. The speed-up in translation units I have tried is between 10 and 20%.

Diff Detail

Event Timeline

djasper updated this revision to Diff 82784.Jan 1 2017, 7:16 AM
djasper retitled this revision from to Remove isIgnored()-test that is more expensive than the analysis behind it.
djasper updated this object.
djasper added a reviewer: rsmith.
djasper added a subscriber: cfe-commits.
rsmith accepted this revision.Jan 2 2017, 2:41 PM
rsmith edited edge metadata.
This revision is now accepted and ready to land.Jan 2 2017, 2:41 PM
djasper closed this revision.Jan 9 2017, 1:22 AM

Landed as r290842.