[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