[PATCH] D134433: [AMDGPU][GISel] Enable Matching of V2S16 G_BUILD_VECTOR
Petar Avramovic via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 29 09:00:33 PDT 2022
Petar.Avramovic added a comment.
You don't want `(<2 x s16>)` but only `s16` `G_FMAXNUM_IEEE`when lowering larger vectors, also you remove `G_BUILD_VECTOR_TRUNC` but use `G_BUILD_VECTOR` instead.
Can you point me to the test case that requires those changes? (You didn't change legalization rules for other `<2 x s16>` instructions).
To be more precise what D134354 <https://reviews.llvm.org/D134354> requires from Legalizer (and pre/post legalizer combiner). Maybe there is simpler way to achieve it.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D134433/new/
https://reviews.llvm.org/D134433
More information about the llvm-commits
mailing list