[PATCH] D155865: [AMDGPU][GlobalIsel] Fix legalizer for G_ABS for odd sized vectors
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 20 11:09:19 PDT 2023
arsenm added inline comments.
================
Comment at: llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp:1214
if (ST.hasVOP3PInsts()) {
getActionDefinitionsBuilder({G_SMIN, G_SMAX, G_UMIN, G_UMAX, G_ABS})
.legalFor({S32, S16, V2S16})
----------------
I don't think there is a v2i16 abs is there? Should these just be lowered?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D155865/new/
https://reviews.llvm.org/D155865
More information about the llvm-commits
mailing list