[PATCH] D28152: Cortex-A57 scheduling model for ARM backend (AArch32)

[Diana Picus via Phabricator via llvm-commits]

rovka added reviewers: javed.absar, kristof.beyls, jmolloy.
rovka added a comment.

Hi,

I don't see anything wrong with this, but I'd like to get a second opinion from someone with more experience in the area.
Also, do you have / can you get any performance results with/without this patch (at least on the test-suite)?

Thanks,
Diana



================
Comment at: test/CodeGen/ARM/cortex-a57-misched-alu.ll:1
+; RUN: llc < %s -mtriple=armv8r-eabi -mcpu=cortex-a57 -enable-misched -verify-misched -debug-only=misched -o - 2>&1 > /dev/null | FileCheck %s
+
----------------
Shouldn't this and all other added test require asserts?


https://reviews.llvm.org/D28152