[llvm] [GISel][AArch64][AMDGPU][RISCV] Canonicalize (sub X, C) -> (add X, -C) (PR #114309)

Madhur Amilkanthwar via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 6 21:37:38 PST 2024


madhur13490 wrote:

Hi @topperc 
Yes, you are right. Your patch is not the culprit.

A glitch in our CI showed a reduced number of fallbacks on the previous days. It is now consistent and shows 28 for our SPEC config (which uses `-mcpu=neoverse-v2+nosve -DSPEC_SUPPRESS_OPENMP -mcpu=neoverse-v2+nosve -O3  -fomit-frame-pointer  -Wl,--allow-multiple-definition -mllvm --global-isel  -mllvm --global-isel-abort=2`)

Thanks for reverting and taking time to build on your end. Apologies for the churn. 

Please commit the patch. 

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


More information about the llvm-commits mailing list