[PATCH] D30265: [APInt] Add APInt::setBits() method to set all bits in range

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 23 07:56:18 PST 2017


RKSimon updated this revision to Diff 89509.
RKSimon added a comment.

Dropped the wrap around functionality as it doesn't appear to have many use cases - and makes it easier to drop in as a replacement for APInt::getLowBitsSet/getHighBitsSet.

Removed use of APInt::setBit and do everything at the word level instead.

Added extra test cases that should cover all mask permutations.


Repository:
  rL LLVM

https://reviews.llvm.org/D30265

Files:
  include/llvm/ADT/APInt.h
  lib/Support/APInt.cpp
  lib/Target/X86/X86ISelLowering.cpp
  lib/Target/X86/X86ShuffleDecodeConstantPool.cpp
  unittests/ADT/APIntTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D30265.89509.patch
Type: text/x-patch
Size: 8251 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170223/673b3f2f/attachment.bin>


More information about the llvm-commits mailing list