[PATCH] D37760: [AMDGPU] do not generate .AMDGPU.config for amdpal os type

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 18 16:13:01 PDT 2017


arsenm added inline comments.


================
Comment at: lib/Target/AMDGPU/AMDGPUAsmPrinter.cpp:254
   MCContext &Context = getObjFileLowering().getContext();
-  if (!STM.isAmdHsaOS()) {
+  if (!STM.isAmdHsaOS() && !STM.isAmdPalOS()) {
     MCSectionELF *ConfigSection =
----------------
Probably should make this a positive check on is mesa (although that might be broken because I'm not sure mesa graphics ever fully switched to explicitly setting the triple)


https://reviews.llvm.org/D37760





More information about the llvm-commits mailing list