[PATCH] D27510: [X86][SSE] Remove AND -> VZEXT combine

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 7 08:48:15 PST 2016


RKSimon added a comment.

In https://reviews.llvm.org/D27510#615866, @spatel wrote:

> LGTM. I'm assuming we have x86 test coverage for this transform somewhere? It'd be good to reference that file(s) in the commit message, so we have a link to those tests.


Thanks Sanjay, yes it still uses tests that were originally added in https://reviews.llvm.org/D7666/https://reviews.llvm.org/rL229480 to support combineVectorZext (or VectorZextCombine as it was known then....).


Repository:
  rL LLVM

https://reviews.llvm.org/D27510





More information about the llvm-commits mailing list