Skip to content

Commit 1b4f428

Browse files
committed
Fix style and format
1 parent 7dce6c6 commit 1b4f428

File tree

2 files changed

+39
-38
lines changed

2 files changed

+39
-38
lines changed

clang/lib/AST/ByteCode/InterpBuiltin.cpp

Lines changed: 16 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -4474,14 +4474,13 @@ bool InterpretBuiltin(InterpState &S, CodePtr OpPC, const CallExpr *Call,
44744474
return interp__builtin_ia32_shuffle_generic(
44754475
S, OpPC, Call, [](unsigned DstIdx, unsigned ShuffleMask) {
44764476
uint8_t Ctlb = static_cast<uint8_t>(ShuffleMask);
4477-
if (Ctlb & 0x80) {
4477+
if (Ctlb & 0x80)
44784478
return std::make_pair(0, -1);
4479-
} else {
4480-
unsigned LaneBase = (DstIdx / 16) * 16;
4481-
unsigned SrcOffset = Ctlb & 0x0F;
4482-
unsigned SrcIdx = LaneBase + SrcOffset;
4483-
return std::make_pair(0, static_cast<int>(SrcIdx));
4484-
}
4479+
4480+
unsigned LaneBase = (DstIdx / 16) * 16;
4481+
unsigned SrcOffset = Ctlb & 0x0F;
4482+
unsigned SrcIdx = LaneBase + SrcOffset;
4483+
return std::make_pair(0, static_cast<int>(SrcIdx));
44854484
});
44864485

44874486
case X86::BI__builtin_ia32_pshuflw:
@@ -4494,9 +4493,9 @@ bool InterpretBuiltin(InterpState &S, CodePtr OpPC, const CallExpr *Call,
44944493
if (LaneIdx < 4) {
44954494
unsigned Sel = (ShuffleMask >> (2 * LaneIdx)) & 0x3;
44964495
return std::make_pair(0, static_cast<int>(LaneBase + Sel));
4497-
} else {
4498-
return std::make_pair(0, static_cast<int>(DstIdx));
44994496
}
4497+
4498+
return std::make_pair(0, static_cast<int>(DstIdx));
45004499
});
45014500

45024501
case X86::BI__builtin_ia32_pshufhw:
@@ -4509,9 +4508,9 @@ bool InterpretBuiltin(InterpState &S, CodePtr OpPC, const CallExpr *Call,
45094508
if (LaneIdx >= 4) {
45104509
unsigned Sel = (ShuffleMask >> (2 * (LaneIdx - 4))) & 0x3;
45114510
return std::make_pair(0, static_cast<int>(LaneBase + 4 + Sel));
4512-
} else {
4513-
return std::make_pair(0, static_cast<int>(DstIdx));
45144511
}
4512+
4513+
return std::make_pair(0, static_cast<int>(DstIdx));
45154514
});
45164515

45174516
case X86::BI__builtin_ia32_pshufd:
@@ -4686,12 +4685,11 @@ bool InterpretBuiltin(InterpState &S, CodePtr OpPC, const CallExpr *Call,
46864685
[](unsigned DstIdx, unsigned Shift) -> std::pair<unsigned, int> {
46874686
unsigned LaneBase = (DstIdx / 16) * 16;
46884687
unsigned LaneIdx = DstIdx % 16;
4689-
if (LaneIdx < Shift) {
4688+
if (LaneIdx < Shift)
46904689
return std::make_pair(0, -1);
4691-
}
4692-
4693-
return std::make_pair(0, static_cast<int>(LaneBase + LaneIdx - Shift));
46944690

4691+
return std::make_pair(0,
4692+
static_cast<int>(LaneBase + LaneIdx - Shift));
46954693
});
46964694

46974695
case X86::BI__builtin_ia32_psrldqi128_byteshift:
@@ -4706,9 +4704,9 @@ bool InterpretBuiltin(InterpState &S, CodePtr OpPC, const CallExpr *Call,
47064704
[](unsigned DstIdx, unsigned Shift) -> std::pair<unsigned, int> {
47074705
unsigned LaneBase = (DstIdx / 16) * 16;
47084706
unsigned LaneIdx = DstIdx % 16;
4709-
if (LaneIdx + Shift < 16) {
4710-
return std::make_pair(0, static_cast<int>(LaneBase + LaneIdx + Shift));
4711-
}
4707+
if (LaneIdx + Shift < 16)
4708+
return std::make_pair(0,
4709+
static_cast<int>(LaneBase + LaneIdx + Shift));
47124710

47134711
return std::make_pair(0, -1);
47144712
});

clang/lib/AST/ExprConstant.cpp

Lines changed: 23 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -12929,16 +12929,16 @@ bool VectorExprEvaluator::VisitCallExpr(const CallExpr *E) {
1292912929
APValue R;
1293012930
if (!evalShuffleGeneric(
1293112931
Info, E, R,
12932-
[](unsigned DstIdx, unsigned ShuffleMask) -> std::pair<unsigned, int> {
12932+
[](unsigned DstIdx,
12933+
unsigned ShuffleMask) -> std::pair<unsigned, int> {
1293312934
uint8_t Ctlb = static_cast<uint8_t>(ShuffleMask);
12934-
if (Ctlb & 0x80) {
12935+
if (Ctlb & 0x80)
1293512936
return std::make_pair(0, -1);
12936-
} else {
12937-
unsigned LaneBase = (DstIdx / 16) * 16;
12938-
unsigned SrcOffset = Ctlb & 0x0F;
12939-
unsigned SrcIdx = LaneBase + SrcOffset;
12940-
return std::make_pair(0, static_cast<int>(SrcIdx));
12941-
}
12937+
12938+
unsigned LaneBase = (DstIdx / 16) * 16;
12939+
unsigned SrcOffset = Ctlb & 0x0F;
12940+
unsigned SrcIdx = LaneBase + SrcOffset;
12941+
return std::make_pair(0, static_cast<int>(SrcIdx));
1294212942
}))
1294312943
return false;
1294412944
return Success(R, E);
@@ -12961,7 +12961,7 @@ bool VectorExprEvaluator::VisitCallExpr(const CallExpr *E) {
1296112961
unsigned Sel = (Mask >> (2 * LaneIdx)) & 0x3;
1296212962
return std::make_pair(0, static_cast<int>(LaneBase + Sel));
1296312963
}
12964-
return std::make_pair(0, static_cast<int>(DstIdx));
12964+
return std::make_pair(0, static_cast<int>(DstIdx));
1296512965
}))
1296612966
return false;
1296712967
return Success(R, E);
@@ -12983,9 +12983,10 @@ bool VectorExprEvaluator::VisitCallExpr(const CallExpr *E) {
1298312983
if (LaneIdx >= HalfSize) {
1298412984
unsigned Rel = LaneIdx - HalfSize;
1298512985
unsigned Sel = (Mask >> (2 * Rel)) & 0x3;
12986-
return std::make_pair(0, static_cast<int>(LaneBase + HalfSize + Sel));
12986+
return std::make_pair(
12987+
0, static_cast<int>(LaneBase + HalfSize + Sel));
1298712988
}
12988-
return std::make_pair(0, static_cast<int>(DstIdx));
12989+
return std::make_pair(0, static_cast<int>(DstIdx));
1298912990
}))
1299012991
return false;
1299112992
return Success(R, E);
@@ -13491,14 +13492,15 @@ bool VectorExprEvaluator::VisitCallExpr(const CallExpr *E) {
1349113492
[](unsigned DstIdx, unsigned Shift) -> std::pair<unsigned, int> {
1349213493
unsigned LaneBase = (DstIdx / 16) * 16;
1349313494
unsigned LaneIdx = DstIdx % 16;
13494-
if (LaneIdx < Shift) {
13495+
if (LaneIdx < Shift)
1349513496
return std::make_pair(0, -1);
13496-
}
13497-
return std::make_pair(0, static_cast<int>(LaneBase + LaneIdx - Shift));
13497+
13498+
return std::make_pair(
13499+
0, static_cast<int>(LaneBase + LaneIdx - Shift));
1349813500
}))
1349913501
return false;
1350013502
return Success(R, E);
13501-
}
13503+
}
1350213504

1350313505
case X86::BI__builtin_ia32_psrldqi128_byteshift:
1350413506
case X86::BI__builtin_ia32_psrldqi256_byteshift:
@@ -13509,14 +13511,15 @@ bool VectorExprEvaluator::VisitCallExpr(const CallExpr *E) {
1350913511
[](unsigned DstIdx, unsigned Shift) -> std::pair<unsigned, int> {
1351013512
unsigned LaneBase = (DstIdx / 16) * 16;
1351113513
unsigned LaneIdx = DstIdx % 16;
13512-
if (LaneIdx + Shift < 16) {
13513-
return std::make_pair(0, static_cast<int>(LaneBase + LaneIdx + Shift));
13514-
}
13515-
return std::make_pair(0, -1);
13514+
if (LaneIdx + Shift < 16)
13515+
return std::make_pair(
13516+
0, static_cast<int>(LaneBase + LaneIdx + Shift));
13517+
13518+
return std::make_pair(0, -1);
1351613519
}))
1351713520
return false;
1351813521
return Success(R, E);
13519-
}
13522+
}
1352013523
case X86::BI__builtin_ia32_vpermi2varq128:
1352113524
case X86::BI__builtin_ia32_vpermi2varpd128: {
1352213525
APValue R;

0 commit comments

Comments
 (0)