Skip to content

Conversation

@chengduoZH
Copy link
Contributor

/paddle/Paddle/paddle/operators/math/sequence_padding_test.cc: In instantiation of ‘void TestSequencePadding(const LoD&, size_t) [with DeviceContext = paddle::platform::CPUDeviceContext; Place = paddle::platform::CPUPlace; T = float; paddle::framework::LoD = std::vector<thrust::host_vector<long unsigned int, thrust::system::cuda::experimental::pinned_allocator<long unsigned int> > >; size_t = long unsigned int]’: /paddle/Paddle/paddle/operators/math/sequence_padding_test.cc:84:66: required from here /paddle/Paddle/paddle/operators/math/sequence_padding_test.cc:34:24: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] for (size_t i = 0; i < cpu_seq.numel(); ++i) { ^ [ 87%] Generating paddle_fluid.dir/linear_chain_crf_op.objdir /paddle/Paddle/paddle/operators/math/sequence_padding_test.cc:72:24: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] for (size_t i = 0; i < cpu_seq.numel(); ++i) { ^ /paddle/Paddle/paddle/operators/math/sequence_padding_test.cc: In instantiation of ‘void TestSequencePadding(const LoD&, size_t) [with DeviceContext = paddle::platform::CUDADeviceContext; Place = paddle::platform::CUDAPlace; T = float; paddle::framework::LoD = std::vector<thrust::host_vector<long unsigned int, thrust::system::cuda::experimental::pinned_allocator<long unsigned int> > >; size_t = long unsigned int]’: /paddle/Paddle/paddle/operators/math/sequence_padding_test.cc:97:67: required from here /paddle/Paddle/paddle/operators/math/sequence_padding_test.cc:34:24: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] for (size_t i = 0; i < cpu_seq.numel(); ++i) { ^ /paddle/Paddle/paddle/operators/math/sequence_padding_test.cc:72:24: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] for (size_t i = 0; i < cpu_seq.numel(); ++i) { ^ 
@chengduoZH chengduoZH requested a review from Xreki January 18, 2018 12:56
Copy link
Contributor

@luotao1 luotao1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@luotao1 luotao1 merged commit 161bd4a into PaddlePaddle:develop Jan 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants