[PATCH] D47736: AMDHSA Code Object v3 assembler syntax update

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: lib/Target/AMDGPU/MCTargetDesc/AMDGPUTargetStreamer.cpp:243-246
+  OS << "\t\t.amdhsa_user_sgpr_dispatch_id "
+     << AMDHSA_BITS_GET(KD.kernel_code_properties,
+                        amdhsa::KERNEL_CODE_PROPERTY_ENABLE_SGPR_DISPATCH_ID)
+     << '\n';
----------------
Can we start omitting entries when they are the default value? One thing that's been bothering me for a while is how much junk always printed out


Repository:
  rL LLVM

https://reviews.llvm.org/D47736