Skip to content

Commit

Permalink
[Autobackout][FunctionalRegression]Revert of change: 1a3dbe9: Fixing …
Browse files Browse the repository at this point in the history
…kernel arg alignment

The change fixes kernel arg alignment seting.
    If a kernel argument has align attribute set then
    the align value is used for initialization.
  • Loading branch information
sys-igc authored and igcbot committed Nov 16, 2024
1 parent 070e5aa commit 0b4b682
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 276 deletions.
2 changes: 0 additions & 2 deletions IGC/Compiler/Optimizer/OpenCLPasses/KernelArgs/KernelArgs.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -102,8 +102,6 @@ alignment_t KernelArg::calcAlignment(const Argument* arg, const DataLayout* DL)
// If we don't need to allocate, we certainly don't need alignment
if (!needsAllocation()) return 0;

if (arg->hasAttribute(llvm::Attribute::Alignment))
return arg->getParamAlign().valueOrOne().value();
Type* typeToAlign = arg->getType();
// Usually, we return the alignment of the parameter type.
// For local pointers, we need the alignment of the *contained* type.
Expand Down
274 changes: 0 additions & 274 deletions IGC/ocloc_tests/SPIRV-Asm/ArgsAlignment.spvasm

This file was deleted.

0 comments on commit 0b4b682

Please sign in to comment.