[PATCH] D75913: [AMDGPU] Use progbits type for .AMDGPU.disasm section

Tony Tye via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 10 10:53:13 PDT 2020


t-tye added a comment.

What does the .AMDGPU.disasm setion contain? If it is effectively metadata then using a note record does not seem unreasonable, but that could simply be another note record in the existing note section, and would need to conform to the note record ABI.

If making this a separate section, it would be good to document in https://llvm.org/docs/AMDGPUUsage.html#sections .


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75913





More information about the llvm-commits mailing list