diff options
author | Jakub Kuderski <kubakuderski@gmail.com> | 2018-07-28 00:54:07 +0000 |
---|---|---|
committer | Jakub Kuderski <kubakuderski@gmail.com> | 2018-07-28 00:54:07 +0000 |
commit | 81db59b6f120a5a0f253fa64fc9f7175314ae29b (patch) | |
tree | d4af22dc88e5ac80b20124a8f48ebce5244fb82e /clang/lib/CodeGen/CodeGenModule.cpp | |
parent | 32fa8715475f2ecff33a0bd5fb92d8aabbf40640 (diff) | |
download | bcm5719-llvm-81db59b6f120a5a0f253fa64fc9f7175314ae29b.tar.gz bcm5719-llvm-81db59b6f120a5a0f253fa64fc9f7175314ae29b.zip |
[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
Diffstat (limited to 'clang/lib/CodeGen/CodeGenModule.cpp')
0 files changed, 0 insertions, 0 deletions