[PATCH] D135147: [GISel] Handle G_TRUNC in `matchExtractVecEltBuildVec`

[Amara Emerson via Phabricator via llvm-commits]

aemerson added inline comments.


================
Comment at: llvm/lib/CodeGen/GlobalISel/CombinerHelper.cpp:3931-3933
   if (!isLegalOrBeforeLegalizer(
           {TargetOpcode::G_BUILD_VECTOR, {SrcTy, SrcTy.getElementType()}}))
     return false;
----------------
Pierre-vh wrote:
> arsenm wrote:
> > I don't see why this legality check (or the one below for build_vector_trunc) are here. The equivalent DAG combine does not check for the legality, and you're not creating a new one in the apply
> @aemerson I see you implemented this one, is it okay to remove the legalizer checks?
Sure, I don’t remember why it’s there either.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D135147/new/

https://reviews.llvm.org/D135147