For G_ABS with type v2s16 and sgpr inputs break down into two s32 G_ABS
instructions.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
llvm/test/CodeGen/AMDGPU/GlobalISel/llvm.abs.vector.ll | ||
---|---|---|
292–293 ↗ | (On Diff #542581) | Why is it legal if there's no vector abs? RegBankSelect doesn't need to consider always illegal operations |
llvm/test/CodeGen/AMDGPU/GlobalISel/llvm.abs.vector.ll | ||
---|---|---|
292–293 ↗ | (On Diff #542581) | Lowering is different |
Comment Actions
lgtm with some nits. Can you fix the patch title to something that isn't "fix"? I don't think anything was wrong here, just sub-optimal
llvm/lib/Target/AMDGPU/AMDGPURegisterBankInfo.cpp | ||
---|---|---|
2427–2435 | indentation looks off | |
llvm/test/CodeGen/AMDGPU/GlobalISel/llvm.abs.vector.ll | ||
5–8 ↗ | (On Diff #543489) | Should just add these cases to the existing llvm.abs.ll tests |
indentation looks off