[PATCH] D25693: [AMDGPU] Emit constant address space data in .rodata section and use relocations instead of fixups (amdhsa only)

[Tony Tye via llvm-commits]

tony-tye added inline comments.


================
Comment at: lib/Target/AMDGPU/Utils/AMDGPUBaseInfo.h:59
+/// target triple \p TT, false otherwise.
+bool emitConstantsToTextSection(const Triple &TT);
+
----------------
Would it be better to name this isEmitConstantsToTextSection or shouldEmitConstantsToTextSection since it does not actually do the emitting?


https://reviews.llvm.org/D25693