[PATCH] D109240: GlobalISel: Artifact combine merge-like opcode and unmerge into copy

[Petar Avramovic via Phabricator via llvm-commits]

Petar.Avramovic added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/GlobalISel/LegalizationArtifactCombiner.h:1081
 
+  bool tryCombineMergeLike(GMergeLikeOp &MI,
+                           SmallVectorImpl<MachineInstr *> &DeadInsts,
----------------
arsenm wrote:
> Should this be merged in with ArtifactValueFinder?
Do you mean whole function or some parts?
To me this looks similar to other tryCombine functions. 
build_vector combines should be very simple, two adjacent instructions starting from build_vector check for elements shuffles. ArtifactValueFinder does advanced look through calculations (mainly through G_INSERT).
@aemerson What do you think?


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

https://reviews.llvm.org/D109240