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

[Aditya Nandakumar via Phabricator via llvm-commits]

aditya_nandakumar added inline comments.


================
Comment at: include/llvm/CodeGen/GlobalISel/LegalizationArtifactCombiner.h:74
+      ;
+
+    // zext(trunc x) - > and (aext/copy/trunc x), mask
----------------
Looks like this patch is not NFC (Is it meant to be?). Looks like the check for isInstUnsupported has been dropped here. Is that intended?


https://reviews.llvm.org/D54116