[PATCH] D28152: Cortex-A57 scheduling model for ARM backend (AArch32)
Diana Picus via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jan 20 03:50:50 PST 2017
rovka added reviewers: javed.absar, kristof.beyls, jmolloy.
rovka added a comment.
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)?
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?
More information about the llvm-commits