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

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 11 08:58:06 PST 2019


fhahn added a comment.

In D56572#1354350 <https://reviews.llvm.org/D56572#1354350>, @evandro wrote:

> 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?


I think it would be good to get a MIR test in from the start as it is easier to see what's going, similar to https://github.com/llvm-mirror/llvm/blob/master/test/CodeGen/AArch64/misched-fusion-crypto-eor.mir .


Repository:
  rL LLVM

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

https://reviews.llvm.org/D56572





More information about the llvm-commits mailing list