[PATCH] D31311: [DAGCombiner] Add vector demanded elements support to ComputeNumSignBits

Filipe Cabecinhas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 31 05:46:53 PDT 2017


filcab accepted this revision.
filcab added a comment.
This revision is now accepted and ready to land.

LGTM. I'd prefer you split this into:

- Commit just changing what is already there (build_vector/extract_vector_elt support)
- Commit adding the INSERT_VECTOR_ELT support

But it's not a strong requirement.

Thank you,
Filipe


Repository:
  rL LLVM

https://reviews.llvm.org/D31311





More information about the llvm-commits mailing list