[PATCH] D157565: [CodeGen] Add AArch64 behavior to existing MFS tests

Thu Aug 17 15:37:06 PDT 2023

rahmanl added inline comments.

================
Comment at: llvm/lib/CodeGen/BasicBlockSections.cpp:294
+        continue;
+      NopInstr.addOperand(
+          MachineOperand::CreateImm(Nop.getOperand(0).getImm()));
----------------
rahmanl wrote:
> Is this because of ARM? Maybe add a comment to explain the rationale.
I see, this is target specific now. Although this code was target-specific before too, but we only wanted code for X86. Do you think we should implement insertNoop (https://llvm.org/doxygen/classllvm_1_1TargetInstrInfo.html#a4ee57d5d6295dfeb44f3b55301b20020) for X86 and AArch64 now?

================
Comment at: llvm/lib/CodeGen/MachineFunctionSplitter.cpp:71

+static cl::opt<bool> AllowUnsupportedTriple(
+    "mfs-split-unsupported-triple",
----------------
Even better naming than mine!

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157565