[PATCH] D28491: [AArch64] Add new subtarget feature to fuse AES crypto operations

[Matthias Braun via Phabricator via llvm-commits]

MatzeB added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64InstrInfo.cpp:1984-1988
+  if (Subtarget.hasAESFusion())
+    // Fuse AES crypto operations.
+    if ((FirstOpcode == AArch64::AESErr && SecondOpcode == AArch64::AESMCrr) ||
+        (FirstOpcode == AArch64::AESDrr && SecondOpcode == AArch64::AESIMCrr))
+      return true;
----------------
MatzeB wrote:
> Have you actually tested this properly? I am pretty sure this function is only called for terminator instructions. I am currently working on bigger rewrites of the macrofusion code to allow fusion inside of basic blocks, I don't think it is possible today.
Just saw that you linked a dependent patch about that.


Repository:
  rL LLVM

https://reviews.llvm.org/D28491