[PATCH] D40146: [JumpThreading] Preservation of DT and LVI across the pass

Brian Rzycki via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 9 14:16:04 PST 2018


brzycki marked 9 inline comments as done.
brzycki added a subscriber: kuba.
brzycki added a comment.

> I'm OK with reapplying this, but when it causes a couple more crashes I'd strongly consider some more exhaustive testing approach.

Thanks @kuba for the review. I agree with you that DDT should have a unittest module and I've started work on it. I have the skeleton in place and am working on the actual tests now. I'm still trying to determine the set of unique CFG shapes/mutations to test for.


https://reviews.llvm.org/D40146





More information about the llvm-commits mailing list