[PATCH] D128344: [AMDGPU] Add the "is_dynamic_callstack" of amd_kernel_code_t to the kernel descriptor

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/include/llvm/Support/AMDHSAKernelDescriptor.h:180-181
   uint32_t compute_pgm_rsrc2;
-  uint16_t kernel_code_properties;
-  uint8_t reserved2[6];
+  uint32_t kernel_code_properties;
+  uint8_t reserved2[4];
 };
----------------
I thought this was already defined here, which was the reason to set it. I'm not sure it makes sense to add the field if it's not already there


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128344