[PATCH] D27521: AArch64 Cortex-A57 FDIV/FSQRT scheduling fix (W-unit)
Renato Golin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Dec 16 08:54:17 PST 2016
rengolin added a comment.
Can you run any benchmark and share relative numbers? Despite the fact that the change is obvious, you may uncover pre-existing bugs in the compiler that need addressing before we can change this. The LLVM test-suite, SPEC, EEMBC, anything would help to have an idea.
Comment at: test/CodeGen/AArch64/arm64-misched-A57-div-fix.ll:5
+; RUN: llc < %s -mtriple=arm64-linux-gnu -mcpu=cortex-a57 -enable-misched -verify-misched -debug-only=misched -o - 2>&1 > /dev/null | not grep A57UnitX
> Tests shouldn't rely on grep, please use FileCheck instead.
It should also check for the unit you want to appear, not just the one you don't. :)
More information about the llvm-commits