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

[Florian Hahn via Phabricator via llvm-commits]

fhahn accepted this revision.
fhahn added a comment.
This revision is now accepted and ready to land.

LGTM, thanks! It would be great if you could extend the test cases a bit more before committing, covering cases with shifted regs and cases where some SUs have multiple successors and not all of them should be fused.



================
Comment at: llvm/lib/Target/AArch64/AArch64MacroFusion.cpp:342
+  // Arithmetic setting flags.
+    case AArch64::ADDSWrr:
+    case AArch64::ADDSXrr:
----------------
This indent is off by one level I think.


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

https://reviews.llvm.org/D56572