[PATCH] D33836: [AArch64] Enable FeatureFuseAES for the generic processor model.

Renato Golin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 2 15:47:05 PDT 2017


rengolin added inline comments.


================
Comment at: test/CodeGen/AArch64/misched-fusion-aes.ll:1
+; RUN: llc %s -o - -mtriple=aarch64-unknown -mcpu=generic -mattr=+crypto | FileCheck %s --check-prefix=CHECK --check-prefix=CHECKCORTEX
 ; RUN: llc %s -o - -mtriple=aarch64-unknown -mcpu=cortex-a53 | FileCheck %s --check-prefix=CHECK --check-prefix=CHECKCORTEX
----------------
javed.absar wrote:
> If the cpu is generic, why is the check 'CHECKCORTEX' ?
> Maybe, I am missing something.
I think it's just repeating the same pattern. Probably good to change to something more meaningful, like "CHECKFUSEAES" or something.


https://reviews.llvm.org/D33836





More information about the llvm-commits mailing list