[PATCH] D111693: [PowerPC] Implement basic instruction macro fusion in Power10

[Jinsong Ji via Phabricator via llvm-commits]

jsji added inline comments.


================
Comment at: llvm/lib/Target/PowerPC/PPC.td:196
+  SubtargetFeature<"fuse-add-logical", "HasAddLogicalFusion", "true",
+                   "Target supports Add with Logical Operations fusion",
+                   [FeatureFusion]>;
----------------
nit: description are not consistent with others, note the position of `Operations`


================
Comment at: llvm/test/CodeGen/PowerPC/macro-fusion.mir:1
+# RUN: llc -mtriple=powerpc64le-unknown-linux-gnu -mcpu=pwr10 -x=mir < %s \
+# RUN:   -debug-only=machine-scheduler -start-before=postmisched 2>&1 \
----------------
Do we new require assert?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111693