[PATCH] D43808: ARM Cortex A57 scheduler fix (missed 16-bit, v8.1/v8.2/v8.3, thumb and pseudo instructions)

Andrew Zhogin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 27 19:56:19 PST 2018


andrew.zhogin added a comment.

In https://reviews.llvm.org/D43808#1020389, @javed.absar wrote:

> LGTM.
>
> By the way, did you  do any bench-marking to see if there were any impact of these changes, although from the changes it looks to me there wont be anything significant, and besides I understand this is simply for CompleteModel.


No, I didn't.
Most of those added instructions from v8.1/v8.2/v8.3. I think only 16-bit VMLA (VMLAH/VMLSH/VNMLAH/VNMLSH) may have some practical meaning.


https://reviews.llvm.org/D43808





More information about the llvm-commits mailing list