[PATCH] D157494: [SystemZ][z/OS] Add support for function name field of PPA1

[Ulrich Weigand via Phabricator via llvm-commits]

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

See minor nit inline.  Otherwise LGTM.



================
Comment at: llvm/lib/Target/SystemZ/SystemZAsmPrinter.cpp:1129
+static void emitPPA1Name(const MachineFunction *MF,
+                         std::unique_ptr<MCStreamer> &OutStreamer) {
+  StringRef OutName = MF->getFunction().getName();
----------------
Please pass `OutStreamer` as first argument for consistency with `emitPPA1Flags`.  Also, just pass in a `StringRef OutName` instead of all of `MF`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157494