[PATCH] D57881: [DTU] Refine the document of mutation APIs [NFC] (PR40528)
Max Kazantsev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Feb 10 21:55:58 PST 2019
mkazantsev added inline comments.
================
Comment at: llvm/include/llvm/Analysis/DomTreeUpdater.h:137
/// CFG in DEBUG mode.
+ /// CAUTION! It is only legal to use this method when you have exactly one
+ /// insertion (and no deletions). This function has to be called *after*
----------------
That looks *highly* fishy. Between which two points we should have exactly one update?
For example, if we use eager strategy and we need to delete 10 edges and insert 10 other edges, is that guaranteed to work if we consecutively make 1 CFG update followed by 1 DT update, and do so for every edge? Can we interpret it as making exactly one update 20 times over?
Is that exactly one update between two flushes for lazy strategy?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D57881/new/
https://reviews.llvm.org/D57881
More information about the llvm-commits
mailing list