This is an archive of the discontinued LLVM Phabricator instance.

[Dominators] Rearrange access specifiers in DominatorTreeBase
ClosedPublic

Authored by kuhar on Jun 22 2017, 1:11 PM.

Details

Summary

This patch makes DominatorTreeBase more readable by putting most important members on top of the class.

Before, the class looked like that: private -> protected (including data members) -> public -> protected.
The patch changes it to: protected (data members only) -> public -> protected -> public.

Diff Detail

Event Timeline

dberlin accepted this revision.Jun 23 2017, 7:22 PM
This revision is now accepted and ready to land.Jun 23 2017, 7:22 PM
kuhar updated this revision to Diff 104071.Jun 26 2017, 6:56 PM

Update the diff.

This revision was automatically updated to reflect the committed changes.