[AMDGPU] Use progbits type for .AMDGPU.disasm section

[Flakebi]

Summary

The note section type implies a specific format that this section does
not have thus tools like readelf fail here. Progbits has no format and
another pipeline compiler already sets the type to progbits.

Konstantin, Mark: Is this going to work on the OpenCL side?

[t-tye]

What does the .AMDGPU.disasm setion contain? If it is effectively metadata then using a note record does not seem unreasonable, but that could simply be another note record in the existing note section, and would need to conform to the note record ABI.

If making this a separate section, it would be good to document in https://llvm.org/docs/AMDGPUUsage.html#sections .

[arsenm]

Can we finally just remove this? There's a steady stream of issues from having this feature at all, and we really don't need it

[nhaehnle]

In D75913#1915455, @arsenm wrote:

Can we finally just remove this? There's a steady stream of issues from having this feature at all, and we really don't need it

I agree. Let's figure this out separately though. This change is a hack (sorry Tony), but I think it's basically fine.

llvm/lib/Target/AMDGPU/AMDGPUAsmPrinter.cpp

OutStreamer->emitRawComment(
       " COMPUTE_PGM_RSRC2:TIDIG_COMP_CNT: " +
       Twine(G_00B84C_TIDIG_COMP_CNT(CurrentProgramInfo.ComputePGMRSrc2)),
       false);
   }
 
   if (DumpCodeInstEmitter) {
 
     OutStreamer->SwitchSection(
-        Context.getELFSection(".AMDGPU.disasm", ELF::SHT_NOTE, 0));
+        Context.getELFSection(".AMDGPU.disasm", ELF::SHT_PROGBITS, 0));
 
     for (size_t i = 0; i < DisasmLines.size(); ++i) {
       std::string Comment = "\n";
       if (!HexLines[i].empty()) {
         Comment = std::string(DisasmLineMaxLen - DisasmLines[i].size(), ' ');
         Comment += " ; " + HexLines[i] + "\n";
       }