[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