[ARM] Use the Cortex-A57 sched model for Cortex-A72

[samparker]

Summary

This mirrors what we already do for AArch64 and LNT scores are improved by a geomean of 1.57%.

[fhahn]

LGTM. AFAIK cortex-a57 and cortex-a72 are close enough for this to be beneficial. AArch64 re-uses the A57 model for A72 too.

[samparker]

Hey Florian,

It was brought to my attention that the scheduler still wasn't enabled because of the missing feature. I've now added this and the geomean improvement is 2.23%. I will shortly add a couple of tests too.

[samparker]

Added the a72 to a couple of scheduling tests, as well as the basic unroll one.

[fhahn]

In D53562#1273812, @samparker wrote:

Hey Florian,

It was brought to my attention that the scheduler still wasn't enabled because of the missing feature. I've now added this and the geomean improvement is 2.23%. I will shortly add a couple of tests too.

So yes, to enable the machine scheduler for a core on ARM, FeatureUseMISched is required. I think it is worth splitting using the Cortex-A57 model and enabling the machine scheduler for Cortex-A72. The A57 model should be more accurate for A72 than no model.

For changing to using the MachineScheduler, the last time I looked at it (about a year ago), I remember seeing some relatively big regressions on some benchmarks, so we might want to be a bit more cautious there, as there might be potential to tweak the scheduling heuristics for ARM. Also, I think it would be good to have numbers for a large set of benchmarks (test-suite + commercial ones)

[samparker]

Ok, fair enough, my LNT numbers show that the MISched results are more variable:

Regressions (%):

benchmark	Model	Model + MISched
Stanford/FloatMM		40.1
McGill/queens	29.6	29.4
Stanford/Puzzle	16.05	15.6
FreeBench/mason/mason		9.27
Olden/power/power		7.61
VersaBench/ecbdes/ecbdes		5.25
BenchmarkGame/fannkuch	6.55
Fhourstones/fhourstones	5.75

Improvements (%):

benchmark	Model	Model + MISched
Stanford/Perm	21.7	22.53
Olden/mst/mst	22.05	22.21
FreeBench/fourinarow/fourinarow	22.83	21.21
TSVC/CrossingThresholds-dbl/CrossingThresholds-dbl	14.17	19.52
TSVC/Searching-flt/Searching-flt	16.26	19.23
TSVC/Searching-dbl/Searching-dbl	16.28	19.15

I will remove the use of the feature for now.

cheers,

llvm/trunk/lib/Target/ARM/ARM.td

def : ProcessorModel<"cortex-a57",  CortexA57Model,     [ARMv8a, ProcA57,
                                                          FeatureHWDivThumb,
                                                          FeatureHWDivARM,
                                                          FeatureCrypto,
                                                          FeatureCRC,
                                                          FeatureFPAO,
                                                          FeatureAvoidPartialCPSR,
                                                          FeatureCheapPredicableCPSR]>;
 
-def : ProcNoItin<"cortex-a72",                          [ARMv8a, ProcA72,
+def : ProcessorModel<"cortex-a72",  CortexA57Model,     [ARMv8a, ProcA72,
                                                          FeatureHWDivThumb,
                                                          FeatureHWDivARM,
                                                          FeatureCrypto,
                                                          FeatureCRC]>;
 
 def : ProcNoItin<"cortex-a73",                          [ARMv8a, ProcA73,
                                                          FeatureHWDivThumb,
                                                          FeatureHWDivARM,

llvm/trunk/test/Transforms/LoopUnroll/ARM/loop-unrolling.ll

 ; RUN: opt -mtriple=armv7 -mcpu=cortex-a57 -loop-unroll -S %s -o - | FileCheck %s --check-prefix=CHECK-UNROLL-A
 ; RUN: opt -mtriple=thumbv7 -mcpu=cortex-a57 -loop-unroll -S %s -o - | FileCheck %s --check-prefix=CHECK-UNROLL-A
+; RUN: opt -mtriple=thumbv7 -mcpu=cortex-a72 -loop-unroll -S %s -o - | FileCheck %s --check-prefix=CHECK-UNROLL-A
 ; RUN: opt -mtriple=thumbv8m -mcpu=cortex-m23 -loop-unroll -S %s -o - | FileCheck %s --check-prefix=CHECK-UNROLL-T1
 ; RUN: opt -mtriple=thumbv8m.main -mcpu=cortex-m33 -loop-unroll -S %s -o - | FileCheck %s --check-prefix=CHECK-UNROLL-T2
 ; RUN: opt -mtriple=thumbv7em -mcpu=cortex-m7 -loop-unroll -S %s -o - | FileCheck %s --check-prefix=CHECK-UNROLL-T2
 
 ; CHECK-LABEL: partial
 define arm_aapcs_vfpcc void @partial(i32* nocapture %C, i32* nocapture readonly %A, i32* nocapture readonly %B) local_unnamed_addr #0 {
 entry:
   br label %for.body