[llvm] [AMDGPU] Print `workgroup_processor_mode` metadata field as a boolean (PR #135627)
Lucas Ramirez via llvm-commits
llvm-commits at lists.llvm.org
Tue Apr 15 03:13:30 PDT 2025
================
@@ -510,7 +510,7 @@ MetadataStreamerMsgPackV4::getHSAKernelProps(const MachineFunction &MF,
if (CodeObjectVersion >= AMDGPU::AMDHSA_COV5 && STM.supportsWGP())
Kern[".workgroup_processor_mode"] =
- Kern.getDocument()->getNode(ProgramInfo.WgpMode);
+ Kern.getDocument()->getNode((bool)ProgramInfo.WgpMode);
----------------
lucas-rami wrote:
Yes I think so too, but all other boolean metadata fields are also stored as `uint32_t` in `SIProgramInfo`. I thought perhaps there was an underlying reason for this I did not see, in any case I did not want to create an exception for this field. If there is really no reason these can all be changed to bool in a future PR.
https://github.com/llvm/llvm-project/pull/135627
More information about the llvm-commits
mailing list