[PATCH] D122747: [NFC][ARM] Tests for Cortex-A57 and Cortex-A72 Fused AES Erratum

Sam Elliott via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 12 11:01:01 PDT 2022


lenary added a comment.

In D122747#3509495 <https://reviews.llvm.org/D122747#3509495>, @lenary wrote:

> I want to keep the number of llvm functions the same, as they cover a wide variety of control flow at a wide variety of data widths which the pass should cope with.
>
> One thing that I would like to cut down is the duplicated set of CHECK lines, one with scheduling and one without, but I'm not sure how to do that (the set with scheduling are there to test that `-mcpu=cortex-a55` enables the pass), I'm not sure of a better way of doing the same.

After a quick look, I can confirm that the ARM Backend seems to disregard `tune-cpu` metadata, the subtarget is initialized with TuneCPU equal to CPU, which makes this difficult. Still investigating whether `--enable-misched=false` will be enough.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D122747/new/

https://reviews.llvm.org/D122747



More information about the llvm-commits mailing list