[llvm] [InstCombine] add control for SimplifyDemandedVectorElts depth limit (PR #113717)

Princeton Ferro via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 7 19:34:34 PST 2024


Prince781 wrote:

Hi @nikic,

> What is the depth cutoff that you plan to use?
1, although we may have other values we'd like to use.

> And do you have a sharable test case demonstrating the compile-time issue?
I'm not able to share anything yet, unfortunately. We have a few internal kernels that see a lot of improvement when this aspect of InstCombine is dialed.

> I'm not opposed to having the option, but at the same time it would be nice to address the issue upstream, as other people may also encounter the issue.
I agree. I'll actually be sending a few more patches to improve InstCombine compile time with vector operations soon.

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


More information about the llvm-commits mailing list