[PATCH] D42717: [JumpThreading] sync DT for LVI analysis (PR 36133)
Brian Rzycki via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Feb 7 13:33:52 PST 2018
brzycki added a comment.
@kuhar and @dberlin I would appreciate any guidance you two may have. I spoke with @sebpop about this problem and we don't see a way around the performance impact to preserve DT for LVI analysis. As it stands now there are cases where the deferred updates change the LVI analysis sufficiently to cause a miscompile. We do think there may be ways to mitigate this loss in the future: one way is to prevent pointless threads where a BB is threaded only to be discarded in a subsequent loop iteration within JumpThreading.
Repository:
rL LLVM
https://reviews.llvm.org/D42717
More information about the llvm-commits
mailing list