From d4721c59f245a2f744aa1f5cd40cb853d61df13a Mon Sep 17 00:00:00 2001 From: Kseniya Tikhomirova Date: Wed, 5 Jul 2023 14:53:19 +0200 Subject: [PATCH] [SYCL] Fix post-commit build failure (#10206) Signed-off-by: Tikhomirova, Kseniya --- sycl/tools/sycl-sanitize/collector.cpp | 2 +- sycl/tools/xpti_helpers/usm_analyzer.hpp | 14 ++++++-------- 2 files changed, 7 insertions(+), 9 deletions(-) diff --git a/sycl/tools/sycl-sanitize/collector.cpp b/sycl/tools/sycl-sanitize/collector.cpp index 64f69af9e6d5c..30cf8e99e38cd 100644 --- a/sycl/tools/sycl-sanitize/collector.cpp +++ b/sycl/tools/sycl-sanitize/collector.cpp @@ -43,7 +43,7 @@ XPTI_CALLBACK_API void xptiTraceInit(unsigned int /*major_version*/, tpCallback); auto &GS = USMAnalyzer::getInstance(); GS.changeTerminationOnErrorState(true); - GS.printToErrorStream(true); + GS.printToErrorStream(); GS.setupUSMHandlers(); } } diff --git a/sycl/tools/xpti_helpers/usm_analyzer.hpp b/sycl/tools/xpti_helpers/usm_analyzer.hpp index e4dd2715e50ce..500ad9a380ce6 100644 --- a/sycl/tools/xpti_helpers/usm_analyzer.hpp +++ b/sycl/tools/xpti_helpers/usm_analyzer.hpp @@ -203,7 +203,7 @@ class USMAnalyzer { TerminateOnError = EnableTermination; } - void printToErrorStream(bool Mode) { PrintToError = true; } + void printToErrorStream() { PrintToError = true; } void setupUSMHandlers() { ArgHandlerPostCall.set_piextUSMHostAlloc(USMAnalyzer::handleUSMHostAlloc); @@ -382,10 +382,9 @@ class USMAnalyzer { static void handleUSMEnqueueFill2D(const pi_plugin &, std::optional, pi_queue, - void *ptr, size_t pitch, - size_t pattern_size, const void *pattern, - size_t width, size_t height, pi_uint32, - const pi_event *, pi_event *) { + void *ptr, size_t pitch, size_t, + const void *, size_t width, size_t height, + pi_uint32, const pi_event *, pi_event *) { // TO DO: add checks for pattern validity CheckPointerValidness("input parameter", ptr, pitch, width, height, "ext_oneapi_fill2d"); @@ -413,9 +412,8 @@ class USMAnalyzer { } static void handleKernelSetArgPointer(const pi_plugin &, - std::optional, - pi_kernel kernel, pi_uint32 arg_index, - size_t arg_size, + std::optional, pi_kernel, + pi_uint32 arg_index, size_t arg_size, const void *arg_value) { // no clarity how to handle complex types so check only simple pointers here if (arg_size == sizeof(arg_value)) {