[llvm] [DFAJumpThreading] Enable DFAJumpThread by default. (PR #157646)

via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 11 03:51:55 PDT 2025


buggfg wrote:

> This pass currently does not have a [maintainer](https://github.com/llvm/llvm-project/blob/main/llvm/Maintainers.md). We should have one if we enable it by default.
> 
> @XChy What is your opinion on the current quality of this pass?
> 
> @buggfg Beyond running SPEC, have you performed any due diligence to ensure this pass is ready for default enablement? Have you performed any targeted fuzzing for this enablement?
> 
> I'll provide new compile-time numbers.

Hi, I’m really glad to hear that both reviewers support enabling DFAJumpThreading by default.  I haven’t had the chance to do any additional research beyond the SPEC2017 benchmarks yet. However, I believe that enabling this optimization by default is essential for the embedded domain, as it will help increase LLVM's impact and uncover other potential issues. :)

https://github.com/llvm/llvm-project/pull/157646


More information about the llvm-commits mailing list