[PATCH] D71629: [SystemZ] Recognize mrecord-mcount in backend

Jonas Paulsson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 17 20:48:31 PST 2019


jonpa updated this revision to Diff 234451.
jonpa added a comment.

Patch updated to call EmitLabel, SwitchSection() and EmitIntValue() instead of emitting raw text.

I don't know why gcc is emitting a "1:" label... Is it OK to instead emit .Ltmp0:, .Ltmp1: labels (any name) for each fentry?

Not exactly sure exactly what type of section should be created, but calling getELFSection() this way gives the right output, it seems.

Switching back to the previous section causes a ".text" to be printed instead of ".previous".

See mrecord-mcount-01.ll for the output...


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

https://reviews.llvm.org/D71629

Files:
  llvm/lib/Target/SystemZ/SystemZAsmPrinter.cpp
  llvm/lib/Target/SystemZ/SystemZISelDAGToDAG.cpp
  llvm/test/CodeGen/SystemZ/mrecord-mcount-01.ll
  llvm/test/CodeGen/SystemZ/mrecord-mcount-02.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D71629.234451.patch
Type: text/x-patch
Size: 4414 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191218/543fcd21/attachment.bin>


More information about the llvm-commits mailing list