[PATCH] D135149: [AMDGPU][GISel] Add missing V2S16 BUILD_VECTOR_TRUNC legalization
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Oct 5 08:09:22 PDT 2022
arsenm accepted this revision.
arsenm added inline comments.
This revision is now accepted and ready to land.
================
Comment at: llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp:2875-2876
+
+ assert(MRI.getType(Src0) == S16);
+ assert(MRI.getType(Src1) == S16);
----------------
These asserts are redundant. The verifier or asserts in buildMerge would catch this being wrong
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D135149/new/
https://reviews.llvm.org/D135149
More information about the llvm-commits
mailing list