[PATCH] D110751: [DomTree] Assert that blocks in queries aren't from another function
Daniil Suchkov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 29 14:46:17 PDT 2021
DaniilSuchkov added inline comments.
================
Comment at: llvm/include/llvm/Support/GenericDomTree.h:353
+ assert((!BB || !BB->getParent() || BB->getParent() == Parent) &&
+ "A node from another function!");
+
----------------
nikic wrote:
> Is `!BB` really a valid input to this function?
I don't know, but since there are no checks for that anywhere and it's not mentioned in its description, I decided refrain from making any assumptions.
If you think it's not, I can add an assertion for that in a separate patch.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D110751/new/
https://reviews.llvm.org/D110751
More information about the llvm-commits
mailing list