[PATCH] D83801: [MC] Pass the section to onSymbolStart() for more context

[James Henderson via Phabricator via llvm-commits]

jhenderson added a comment.

In D83801#2152452 <https://reviews.llvm.org/D83801#2152452>, @rochauha wrote:

> In D83801#2152421 <https://reviews.llvm.org/D83801#2152421>, @MaskRay wrote:
>
> > Can you demonstrate how you are going to use the new interface?
>
>
> The note section in AMDGPU ELF objects contains metadata. But we need to know whether the section is note section to deal with it.


I'm not sure I follow. Are you saying you need information from the note section to disassemble other sections? Or are you saying you need it to disassemble the note section? If the former, there's already code in the ELFDumper of llvm-readobj for parsing AMDGPU note sections, which you could reuse (it would need moving somewhere outside the llvm-readobj tool into one of the libraries, maybe in the ELFObjectFile stuff, I'm not sure). If the latter, there's still opposition to adding the functionality to the disassembler (the previous concerns haven't been addressed), given that llvm-readobj already provides this functionality.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83801