[PATCH] D71093: [InstCombine] Insert instructions before adding them to worklist
Jakub Kuderski via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Dec 6 08:20:12 PST 2019
kuhar added a comment.
> This looks fine, I wonder whether it's worth testing [how easy it's to test, that is]
I'd rather not create that rely on debug messages. As an alternative, we could add an assertion in `Worklist::Add` checking if the added instruction has a parent. What do you think?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D71093/new/
https://reviews.llvm.org/D71093
More information about the llvm-commits
mailing list