[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