[PATCH] D30614: [APInt] Move operator~ out of line to make it better able to reused memory allocation from temporary objects

[Craig Topper via Phabricator via llvm-commits]

craig.topper added a comment.

It is used in other places for example

lib/CodeGen/SelectionDAG/DAGCombiner.cpp
2017:    if (DAG.MaskedValueIsZero(N1, ~APInt::getSignBit(BitWidth))) {
8127:    SignMask = ~APInt::getSignBit(SourceVT.getSizeInBits());
10226:        SignMask = ~APInt::getSignBit(N0.getScalarValueSizeInBits());
10230:        SignMask = ~APInt::getSignBit(IntVT.getSizeInBits());
11609:  APInt Mask = ~APInt::getBitsSet(IVal.getValueSizeInBits(),

I'll see if I can construct a test case for the move behavior.


https://reviews.llvm.org/D30614