Should recent Apple CPUs use the new model instead of `CycloneModel`?

[dianqk]

We recently discovered a performance issue with the apple m1 series in rust-lang/rust#126937 (comment). I believe this is due to EarlyIfConversion using an outdated mispredict penalty. The mispredict penalty from apple-a7 to apple-m4 uses CycloneModel's definition of 16, which I assume is an outdated value?

cc @fhahn @TNorthover

[llvmbot]

@llvm/issue-subscribers-backend-aarch64

Author: DianQK (DianQK)

We recently discovered a performance issue with the apple m1 series in https://github.com/rust-lang/rust/issues/126937#issuecomment-2226834122. I believe this is due to EarlyIfConversion using an outdated mispredict penalty. The mispredict penalty from apple-a7 to apple-m4 uses `CycloneModel`'s definition of 16, which I assume is an outdated value?

cc @fhahn @TNorthover

[GuillaumePressiat]

Thank you @dianqk.

Has anybody take a look on this?

You adressed it to: @fhahn @TNorthover (mention cc); in case it can helps.

Thanks