[PATCH] D12680: [InstCombine] Added vector demanded bits support for SSE4A EXTRQ/INSERTQ instructions

Simon Pilgrim via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 17 13:40:08 PDT 2015


RKSimon added a comment.

In http://reviews.llvm.org/D12680#247968, @spatel wrote:

> Can we hoist the getVectorNumElements() and associated assert into the helper as well? If yes, please do. And certainly now that we have a helper, let's use it to reduce the code bloat wherever possible.


Thanks Sanjay, I'll look at hoisting the getVectorNumElements() when I update the other functions to use the helper - probably over the weekend.


Repository:
  rL LLVM

http://reviews.llvm.org/D12680





More information about the llvm-commits mailing list