[PATCH] D47566: AMDHSA: Code object v3 updates

[Scott Linder via Phabricator via llvm-commits]

scott.linder added inline comments.


================
Comment at: lib/Target/AMDGPU/MCTargetDesc/AMDGPUTargetStreamer.cpp:381
+  Streamer.PushSection();
+  Streamer.SwitchSection(&ReadOnlySection);
+
----------------
Should this switch be happening here? Shouldn't the assembly writer be able to put this descriptor in any section?


https://reviews.llvm.org/D47566