[PATCH] D62338: [globalisel][legalizer] Combine G_TRUNC+G_MERGE_VALUES in artifact combiner

[Daniel Sanders via Phabricator via llvm-commits]

dsanders marked an inline comment as done.
dsanders added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/GlobalISel/LegalizationArtifactCombiner.h:349
+        return true;
+      } else if (DstTy.isScalar() && OriginTy.isScalar() &&
+                 DstTy.getSizeInBits() < OriginTy.getSizeInBits()) {
----------------
arsenm wrote:
> Aren't these both required to be scalars anyway? If a vector is involved you have to use G_CONCAT_VECTOR or G_BUILD_VECTOR
Good point. I'd forgotten we had split vectors out


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D62338