[PATCH] D81894: [llvm-readobj][COFF] add .llvm.call-graph-profile section dump

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 19 14:42:10 PDT 2020


MaskRay added a comment.

In D81894#2104538 <https://reviews.llvm.org/D81894#2104538>, @zequanwu wrote:

> In D81894#2104228 <https://reviews.llvm.org/D81894#2104228>, @MaskRay wrote:
>
> > In D81894#2104203 <https://reviews.llvm.org/D81894#2104203>, @zequanwu wrote:
> >
> > > Add comments to explain what byte range represents in test.
> >
> >
> > I second the idea that assembly with `.byte, .short, .long` may improve readability.
>
>
> Then I need to create a section named `.llvm.call-graph-profile`, but section name cannot have the token `-`.


D82240 <https://reviews.llvm.org/D82240>


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81894





More information about the llvm-commits mailing list