[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