[PATCH] D140883: [AMDGPU] Simplify getNumFlatOffsetBits. NFC.
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jan 3 06:35:30 PST 2023
arsenm added inline comments.
================
Comment at: llvm/lib/Target/AMDGPU/SIInstrInfo.cpp:7935
- bool Signed = FlatVariant != SIInstrFlags::FLAT;
+ bool AllowNegative = FlatVariant != SIInstrFlags::FLAT;
if (ST.hasNegativeScratchOffsetBug() &&
----------------
I don't really like the AllowNegative naming. SignBitIgnored?
================
Comment at: llvm/lib/Target/AMDGPU/Utils/AMDGPUBaseInfo.cpp:2507-2510
if (AMDGPU::isGFX10(ST))
- return Signed ? 12 : 11;
+ return 12;
+ return 13;
----------------
Can return ternary operator
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D140883/new/
https://reviews.llvm.org/D140883
More information about the llvm-commits
mailing list