[PATCH] D32293: InstructionSimplify: Simplify a shuffle with a undef mask to undef
Zvi Rackover via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Apr 20 09:07:42 PDT 2017
zvi added inline comments.
================
Comment at: test/Transforms/InstSimplify/shufflevector.ll:113-119
define <4 x i32> @undef_mask(<4 x i32> %x) {
; CHECK-LABEL: @undef_mask(
; CHECK-NEXT: ret <4 x i32> undef
;
%shuf = shufflevector <4 x i32> %x, <4 x i32> undef, <4 x i32> undef
ret <4 x i32> %shuf
}
----------------
davide wrote:
> We have a similar case here, maybe it makes sense to handle them together?
Sorry i don't follow you. What do you mean by handling them together?
This case you are pointing out is handled by a more generic combine starting from DAGCombine.cpp:4154
Repository:
rL LLVM
https://reviews.llvm.org/D32293
More information about the llvm-commits
mailing list