[PATCH] D56572: [AArch64] Add new target feature to fuse arithmetic and logic operations

[Evandro Menezes via Phabricator via llvm-commits]

evandro marked 2 inline comments as done.
evandro added a comment.

In D56572#1354333 <https://reviews.llvm.org/D56572#1354333>, @fhahn wrote:

> Would it be possible to have a MIR test here, which just runs the scheduler?


I tried to keep the existing pattern, but I could add one too.  Should it be done now or later?



================
Comment at: llvm/lib/Target/AArch64/AArch64Subtarget.h:324
+           hasFuseAES() || hasFuseArithmeticLogic() ||
+           hasFuseCCSelect() || hasFuseLiterals();
   }
----------------
fhahn wrote:
> Not related to this patch directly, but is there a reason hasFuseCryptoEOR() is missing here?
Since Cyclone is the only target using it, I guess that it doesn't care about running the `MachineScheduler`.  At least not in the public repository.  Though in the associated test case it is enabled in the command line.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D56572