[llvm] [DFAJumpThreading] Enable the pass by default (PR #83033)
Usman Nadeem via llvm-commits
llvm-commits at lists.llvm.org
Tue Apr 9 15:56:46 PDT 2024
UsmanNadeem wrote:
> The performance improvement brought by DFAJumpThreading looks fairly good to me. But it's uncertain whether we test this pass enough for correctness, especially with just few maintainance after the initial patch of DFAJumpThreading. To add it into the pipeline, I believe we need more correctness proofs apart from internal unit tests.
I believe that enabling it by default will enable more testing, and thus uncover more bugs. We have plenty of time till the next release to fix them.
https://github.com/llvm/llvm-project/pull/83033
More information about the llvm-commits
mailing list