shylie/llvm-project
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[DAG] visitTRUNCATE - early out from computeKnownBits/ComputeNumSignBits failures. NFC. (#154111)

Browse Source 
Avoid unnecessary (costly) computeKnownBits/ComputeNumSignBits calls - use MaskedValueIsZero instead of computeKnownBits directly to simplify code.
This commit is contained in:
Simon Pilgrim 2025-08-18 14:55:09 +01:00 committed by GitHub
parent 81c06d198e
commit 858d1dfa2c
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 6 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
View File

@ -16332,25 +16332,22 @@ SDValue DAGCombiner::visitTRUNCATE(SDNode *N) {
// (trunc (abdu/abds a, b)) -> (abdu/abds (trunc a), (trunc b))
if ((!LegalOperations || N0.hasOneUse()) &&
TLI.isOperationLegal(N0.getOpcode(), VT)) {
EVT SrcVT = N0.getValueType();
EVT TruncVT = VT;
unsigned SrcBits = SrcVT.getScalarSizeInBits();
unsigned TruncBits = TruncVT.getScalarSizeInBits();
unsigned NeededBits = SrcBits - TruncBits;
SDValue A = N0.getOperand(0);
SDValue B = N0.getOperand(1);
bool CanFold = false;
if (N0.getOpcode() == ISD::ABDU) {
KnownBits KnownA = DAG.computeKnownBits(A);
KnownBits KnownB = DAG.computeKnownBits(B);
CanFold = KnownA.countMinLeadingZeros() >= NeededBits &&
KnownB.countMinLeadingZeros() >= NeededBits;
APInt UpperBits = APInt::getBitsSetFrom(SrcBits, TruncBits);
CanFold = DAG.MaskedValueIsZero(B, UpperBits) &&
DAG.MaskedValueIsZero(A, UpperBits);
} else {
unsigned SignBitsA = DAG.ComputeNumSignBits(A);
unsigned SignBitsB = DAG.ComputeNumSignBits(B);
CanFold = SignBitsA > NeededBits && SignBitsB > NeededBits;
unsigned NeededBits = SrcBits - TruncBits;
CanFold = DAG.ComputeNumSignBits(B) > NeededBits &&
DAG.ComputeNumSignBits(A) > NeededBits;
}
if (CanFold) {