Don't bother pre-shrinking "s_cmp_lg_u32 reg, 0" to s_cmpk_lg_u32
because 0 is already an inline constant so the s_cmpk form is no
smaller.
This is just for consistency with the surrounding code and to simplify a
downstream patch.
Paths
| Differential D138993
[AMDGPU] Use s_cmp instead of s_cmpk ClosedPublic Authored by foad on Nov 30 2022, 2:42 AM.
Details
Summary Don't bother pre-shrinking "s_cmp_lg_u32 reg, 0" to s_cmpk_lg_u32 This is just for consistency with the surrounding code and to simplify a
Diff Detail
Event TimelineThis revision is now accepted and ready to land.Nov 30 2022, 9:09 AM Closed by commit rG3d9e226081cf: [AMDGPU] Use s_cmp instead of s_cmpk (authored by foad). · Explain WhyNov 30 2022, 10:04 AM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 479004 llvm/lib/Target/AMDGPU/SIISelLowering.cpp
llvm/test/CodeGen/AMDGPU/expand-scalar-carry-out-select-user.ll
llvm/test/CodeGen/AMDGPU/udiv.ll
|