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

Ulrich Weigand via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 18 10:47:47 PST 2019


uweigand added a comment.

In D71629#1789965 <https://reviews.llvm.org/D71629#1789965>, @jonpa wrote:

> Patch updated per review.
>
> Beginning to understand how this is supposed to work now, but still not sure why the text assembly looks ok, while I see with objdump --filetype=obj the new section filled with zeros:
>
>   llc llvm/test/CodeGen/SystemZ/mrecord-mcount-01.ll -mtriple=s390x-linux-gnu -mcpu=z10 -o out.o --filetype=obj; objdump -D -z out.o
>  
>   0000000000000000 <__mcount_loc>:
>   0:   00 00 00 00             .long   0x00000000
>   4:   00 00 00 00             .long   0x00000000
>   8:   00 00 00 00             .long   0x00000000
>   c:   00 00 00 00             .long   0x00000000
>  
>
>
> I was expecting to see something like <test1> and <test2> somwhere there... Will the linker fix this somehow (not sure how it could since it's just zeroes...)?


Yes, this will be done via relocations.  You should add -r to the objdump command line, then you'll see them :-)


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

https://reviews.llvm.org/D71629





More information about the llvm-commits mailing list