[PATCH] D54116: [GlobalISel] Refactor the artifact combiner a bit by using MIPatternMatch

Volkan Keles via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 5 11:52:04 PST 2018


volkan marked an inline comment as done.
volkan added a comment.

In https://reviews.llvm.org/D54116#1287691, @aditya_nandakumar wrote:

> Thanks. With this patch, are there any more users of getOpcodeDef? If not, we should probably remove that helper.


Yes, it is used by tryFoldImplicitDef and tryCombineMerges. We can remove it once we have matchers for G_IMPLICIT_DEF and G_MERGE_VALUES.


https://reviews.llvm.org/D54116





More information about the llvm-commits mailing list