[PATCH] AMDGPU/SI: Emit amd_kernel_code_t in EmitFunctionBodyStart()

Tom Stellard thomas.stellard at amd.com
Thu Jun 25 06:54:11 PDT 2015


================
Comment at: lib/Target/AMDGPU/AMDGPUAsmPrinter.cpp:119
@@ -118,3 @@
-    EmitAmdKernelCodeT(MF, KernelInfo);
-    OutStreamer->EmitCodeAlignment(2 << (MF.getAlignment() - 1));
-  } else if (STM.getGeneration() >= AMDGPUSubtarget::SOUTHERN_ISLANDS) {
----------------
arsenm wrote:
> Is this alignment  not required anymore?
Alignment for the function is emitted in EmitFunctionHeader(). We don't need to re-emit it after the AMDKernelCodeT object, because the object is 256 bytes (which is the same as the alignment), so the start of the function, is already aligned properly.

http://reviews.llvm.org/D10705

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list