[PATCH] D80713: [AMDGPU] Support disassembly for AMDGPU kernel descriptors
Madhur Amilkanthwar via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 22 06:57:45 PDT 2020
madhur13490 added inline comments.
================
Comment at: llvm/lib/Target/AMDGPU/Disassembler/AMDGPUDisassembler.cpp:1255
+
+MCDisassembler::DecodeStatus AMDGPUDisassembler::decodeKernelDescriptor(
+ StringRef KdName, ArrayRef<uint8_t> Bytes, uint64_t KdAddress) const {
----------------
rochauha wrote:
> madhur13490 wrote:
> > Can you please have function in reverse order? E.g."onSymbolStart" calls "decodeKernelDescriptor" so latter should be above former. This way it's readable and and at a predictable location. Ideally, callees should be above callers.
> I was under the impression that having the current order of functions helps to 'incrementally zoom' into the details while reading the code.
No, You can see other files e.g. runOn* functions are always at the bottom. It is also important for compiler as it would see declarations/definitions before callers.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D80713/new/
https://reviews.llvm.org/D80713
More information about the llvm-commits
mailing list