diff --git a/sycl/include/sycl/ext/intel/experimental/pipes.hpp b/sycl/include/sycl/ext/intel/experimental/pipes.hpp index 7dedc1d4cbc26..1d73e496e4ef5 100644 --- a/sycl/include/sycl/ext/intel/experimental/pipes.hpp +++ b/sycl/include/sycl/ext/intel/experimental/pipes.hpp @@ -68,6 +68,9 @@ class pipe : public pipe_base { // Host API static _dataT read(queue &Q, bool &Success, memory_order Order = memory_order::seq_cst) { + // Order is currently unused. + std::ignore = Order; + const device Dev = Q.get_device(); bool IsPipeSupported = Dev.has_extension("cl_intel_program_scope_host_pipe"); @@ -95,6 +98,9 @@ class pipe : public pipe_base { static void write(queue &Q, const _dataT &Data, bool &Success, memory_order Order = memory_order::seq_cst) { + // Order is currently unused. + std::ignore = Order; + const device Dev = Q.get_device(); bool IsPipeSupported = Dev.has_extension("cl_intel_program_scope_host_pipe"); @@ -221,6 +227,9 @@ class pipe : public pipe_base { // Host API static _dataT read(queue &Q, memory_order Order = memory_order::seq_cst) { + // Order is currently unused. + std::ignore = Order; + const device Dev = Q.get_device(); bool IsPipeSupported = Dev.has_extension("cl_intel_program_scope_host_pipe"); @@ -241,6 +250,9 @@ class pipe : public pipe_base { static void write(queue &Q, const _dataT &Data, memory_order Order = memory_order::seq_cst) { + // Order is currently unused. + std::ignore = Order; + const device Dev = Q.get_device(); bool IsPipeSupported = Dev.has_extension("cl_intel_program_scope_host_pipe"); diff --git a/sycl/unittests/pipes/host_pipe_registration.cpp b/sycl/unittests/pipes/host_pipe_registration.cpp index dd18d98d864d1..4b5b4520e6d02 100644 --- a/sycl/unittests/pipes/host_pipe_registration.cpp +++ b/sycl/unittests/pipes/host_pipe_registration.cpp @@ -100,6 +100,7 @@ pi_result after_piDeviceGetInfo(pi_device device, pi_device_info param_name, switch (param_name) { case PI_DEVICE_INFO_EXTENSIONS: { if (param_value) { + std::ignore = param_value_size; assert(param_value_size >= sizeof(MockSupportedExtensions)); std::memcpy(param_value, MockSupportedExtensions, sizeof(MockSupportedExtensions)); @@ -153,10 +154,10 @@ TEST_F(PipeTest, Basic) { // Testing read int HostPipeReadData; HostPipeReadData = Pipe::read(q); - assert(HostPipeReadData == PipeReadVal); + EXPECT_EQ(HostPipeReadData, PipeReadVal); // Testing write int HostPipeWriteData = 9; Pipe::write(q, HostPipeWriteData); - assert(PipeWriteVal == 9); + EXPECT_EQ(PipeWriteVal, 9); }