[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