[PATCH] D123343: [AMDGPU] Refactor LDS alignment checks.
Jay Foad via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 13 06:56:44 PDT 2022
foad added inline comments.
================
Comment at: llvm/lib/Target/AMDGPU/SIISelLowering.cpp:1555
// gfx8 and older.
- bool AlignedBy16 = Alignment >= Align(16);
- if (IsFast)
- *IsFast = AlignedBy16;
+ RequiredAlignment = Align(16);
+ break;
----------------
You don't need this - it is already handled by using PowerOf2Ceil to initialize RequiredAlignment.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D123343/new/
https://reviews.llvm.org/D123343
More information about the llvm-commits
mailing list