[llvm] 315bd96 - Use std::make_tuple instead initializer list
Lei Zhang via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 22 12:45:33 PDT 2020
Author: Lei Zhang
Date: 2020-06-22T15:43:40-04:00
New Revision: 315bd964377fe8a465e2f6c8c7c06ae9c900bb0f
URL: https://github.com/llvm/llvm-project/commit/315bd964377fe8a465e2f6c8c7c06ae9c900bb0f
DIFF: https://github.com/llvm/llvm-project/commit/315bd964377fe8a465e2f6c8c7c06ae9c900bb0f.diff
LOG: Use std::make_tuple instead initializer list
Hopefully this pleases GCC-5 and fixes the build error:
LowerExpectIntrinsic.cpp:62:53: error: converting to
'std::tuple<unsigned int, unsigned int>' from initializer list would use
explicit constructor 'constexpr std::tuple<_T1, _T2>::tuple(_U1&&,
_U2&&) [with _U1 = llvm::cl::opt<unsigned int>&; _U2 =
llvm::cl::opt<unsigned int>&; <template-parameter-2-3> = void; _T1 =
unsigned int; _T2 = unsigned int]'
return {LikelyBranchWeight, UnlikelyBranchWeight};
Differential Revision: https://reviews.llvm.org/D82325
Added:
Modified:
llvm/lib/Transforms/Scalar/LowerExpectIntrinsic.cpp
Removed:
################################################################################
diff --git a/llvm/lib/Transforms/Scalar/LowerExpectIntrinsic.cpp b/llvm/lib/Transforms/Scalar/LowerExpectIntrinsic.cpp
index 4b98be28b862..05db70c787bb 100644
--- a/llvm/lib/Transforms/Scalar/LowerExpectIntrinsic.cpp
+++ b/llvm/lib/Transforms/Scalar/LowerExpectIntrinsic.cpp
@@ -59,7 +59,8 @@ std::tuple<uint32_t, uint32_t> getBranchWeight(Intrinsic::ID IntrinsicID,
CallInst *CI, int BranchCount) {
if (IntrinsicID == Intrinsic::expect) {
// __builtin_expect
- return {LikelyBranchWeight, UnlikelyBranchWeight};
+ return std::make_tuple(LikelyBranchWeight.getValue(),
+ UnlikelyBranchWeight.getValue());
} else {
// __builtin_expect_with_probability
assert(CI->getNumOperands() >= 3 &&
@@ -71,7 +72,7 @@ std::tuple<uint32_t, uint32_t> getBranchWeight(Intrinsic::ID IntrinsicID,
double FalseProb = (1.0 - TrueProb) / (BranchCount - 1);
uint32_t LikelyBW = ceil((TrueProb * (double)(INT32_MAX - 1)) + 1.0);
uint32_t UnlikelyBW = ceil((FalseProb * (double)(INT32_MAX - 1)) + 1.0);
- return {LikelyBW, UnlikelyBW};
+ return std::make_tuple(LikelyBW, UnlikelyBW);
}
}
More information about the llvm-commits
mailing list