[PATCH] D77953: GlobalISel: Fix casted unmerge of G_CONCAT_VECTORS
Amara Emerson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 13 17:25:13 PDT 2020
aemerson accepted this revision.
aemerson added a comment.
This revision is now accepted and ready to land.
LGTM with nits.
================
Comment at: llvm/include/llvm/CodeGen/GlobalISel/LegalizationArtifactCombiner.h:469
+
+ // Handle split to smaller vectors, with converssions.
+ // %2(<8 x s8>) = G_CONCAT_VECTORS %0(<4 x s8>), %1(<4 x s8>)
----------------
typo converssions
================
Comment at: llvm/include/llvm/Support/LowLevelTypeImpl.h:150
+ LLT divide(int Factor) const {
+ assert(Factor != 1);
+ if (isVector())
----------------
Can we assert that the type is divisible by Factor?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D77953/new/
https://reviews.llvm.org/D77953
More information about the llvm-commits
mailing list