未验证 提交 076c41ef 编写于 作者: W Wang Xin 提交者: GitHub

fix uninitialized, tautological-constant-out-of-range-compare and...

fix uninitialized, tautological-constant-out-of-range-compare and literal-conversion warning on macos (#47341)
上级 dfe6d8fa
...@@ -215,8 +215,13 @@ if(APPLE) ...@@ -215,8 +215,13 @@ if(APPLE)
endif() endif()
# On Mac OS X register class specifier is deprecated and will cause warning error on latest clang 10.0 # On Mac OS X register class specifier is deprecated and will cause warning error on latest clang 10.0
set(COMMON_FLAGS set(COMMON_FLAGS
-Wno-deprecated-register -Werror=format -Wno-deprecated-register
-Werror=inconsistent-missing-override -Werror=braced-scalar-init) -Werror=format
-Werror=inconsistent-missing-override
-Werror=braced-scalar-init
-Werror=uninitialized
-Werror=tautological-constant-out-of-range-compare
-Werror=literal-conversion)
endif() endif()
if(WITH_HETERPS AND WITH_PSLIB) if(WITH_HETERPS AND WITH_PSLIB)
......
...@@ -37,7 +37,7 @@ class ThreadPoolTempl { ...@@ -37,7 +37,7 @@ class ThreadPoolTempl {
: env_(env), : env_(env),
allow_spinning_(allow_spinning), allow_spinning_(allow_spinning),
always_spinning_(always_spinning), always_spinning_(always_spinning),
global_steal_partition_(EncodePartition(0, num_threads_)), global_steal_partition_(EncodePartition(0, num_threads)),
blocked_(0), blocked_(0),
num_tasks_(0), num_tasks_(0),
done_(false), done_(false),
......
...@@ -328,7 +328,8 @@ static void FillConstantLike(const VariableWrapper &ref_var, ...@@ -328,7 +328,8 @@ static void FillConstantLike(const VariableWrapper &ref_var,
// grad mission // grad mission
// we can't get data_type_ directly. We need to check if we can only use // we can't get data_type_ directly. We need to check if we can only use
// default data_type for now. // default data_type for now.
if (ref_var.ForwardDataType() != -1) { if (ref_var.ForwardDataType() !=
static_cast<framework::proto::VarType::Type>(-1)) {
dst_tensor->mutable_data( dst_tensor->mutable_data(
place, framework::TransToPhiDataType(ref_var.ForwardDataType())); place, framework::TransToPhiDataType(ref_var.ForwardDataType()));
} else { } else {
......
...@@ -502,7 +502,7 @@ HOSTDEVICE inline T cubic_convolution2(T x, T A) { ...@@ -502,7 +502,7 @@ HOSTDEVICE inline T cubic_convolution2(T x, T A) {
template <typename T> template <typename T>
HOSTDEVICE inline void get_cubic_upsample_coefficients(T coeffs[4], T t) { HOSTDEVICE inline void get_cubic_upsample_coefficients(T coeffs[4], T t) {
T A = -0.75; T A = static_cast<T>(-0.75);
T x1 = t; T x1 = t;
coeffs[0] = cubic_convolution2<T>(x1 + 1.0, A); coeffs[0] = cubic_convolution2<T>(x1 + 1.0, A);
......
...@@ -128,7 +128,7 @@ void LstsqKernel(const Context& dev_ctx, ...@@ -128,7 +128,7 @@ void LstsqKernel(const Context& dev_ctx,
// run once the driver, first to get the optimal workspace size // run once the driver, first to get the optimal workspace size
int lwork = -1; int lwork = -1;
T wkopt; T wkopt = 0.0;
ValueType rwkopt; ValueType rwkopt;
int iwkopt = 0; int iwkopt = 0;
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册