[Dominators] Make applyUpdate's documentation less confusing [NFC]
authorJakub Kuderski <kubakuderski@gmail.com>
Sat, 28 Jul 2018 00:54:07 +0000 (00:54 +0000)
committerJakub Kuderski <kubakuderski@gmail.com>
Sat, 28 Jul 2018 00:54:07 +0000 (00:54 +0000)
commit81db59b6f120a5a0f253fa64fc9f7175314ae29b
treed4af22dc88e5ac80b20124a8f48ebce5244fb82e
parent32fa8715475f2ecff33a0bd5fb92d8aabbf40640
[Dominators] Make applyUpdate's documentation less confusing [NFC]

Summary:
It was pointed out by @chandlerc that it's not clear whether both applyUpdates and insert/deleteEdge can be used to perform multiple updates.

IMO, the confusing part was that the comment above applyUpdates made a comparison of expected update time between calling it and calling insert/deleteEdge multiple times. It's generally not possible to safely call insert/deleteEdge multiple times, which documentation for each of the 3 functions warns about, so the whole comparison makes very little sense. On top of that, the comment is already lengthy, so I think it's best to just get rid of this comparison.

Reviewers: chandlerc, asbirlea, NutshellySima, grosser

Reviewed By: chandlerc

Subscribers: llvm-commits, chandlerc

Differential Revision: https://reviews.llvm.org/D49944

llvm-svn: 338184
llvm/include/llvm/Support/GenericDomTree.h