This is an archive of the discontinued LLVM Phabricator instance.

[CFG/BasicBlock] Rename pred_const to const_pred. [NFC]
ClosedPublic

Authored by asbirlea on Mar 10 2020, 3:31 PM.

Details

Summary

Rename pred_const_range to const_pred_range to make it consistent with
the other pred/succ iterator definitions: for successors this file defines const_succ_range; the predecessors constant iterators also use const as the first token: const_pred_iterator; there is also a definition in clang for const_pred_range.

Diff Detail

Event Timeline

asbirlea created this revision.Mar 10 2020, 3:31 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 10 2020, 3:31 PM
dblaikie accepted this revision.Mar 12 2020, 10:53 AM

I don't mind too strongly either way - but if you want to commit this, could you mention in the commit message which other types this is consistent with?

This revision is now accepted and ready to land.Mar 12 2020, 10:53 AM
asbirlea edited the summary of this revision. (Show Details)Mar 25 2020, 11:50 AM
This revision was automatically updated to reflect the committed changes.