From 4727d3ce9cc3c28d2752386cfc54ddd0b3bf260a Mon Sep 17 00:00:00 2001
From: "Chereshnev, Eugene" <eugene.chereshnev@intel.com>
Date: Tue, 21 Jan 2025 13:21:05 -0800
Subject: [PATCH] xe: conv: clean up log messages

---
 src/gpu/intel/jit/conv/tiler.cpp          | 3 +--
 src/gpu/intel/jit/ir/send_plan.cpp        | 2 +-
 src/gpu/intel/jit/pooling/gen_pooling.cpp | 2 --
 src/gpu/intel/jit/reorder/gen_reorder.cpp | 2 --
 src/gpu/intel/jit/utils/trace.cpp         | 3 +--
 5 files changed, 3 insertions(+), 9 deletions(-)

diff --git a/src/gpu/intel/jit/conv/tiler.cpp b/src/gpu/intel/jit/conv/tiler.cpp
index c98fccf0aea..db761818f3e 100644
--- a/src/gpu/intel/jit/conv/tiler.cpp
+++ b/src/gpu/intel/jit/conv/tiler.cpp
@@ -1563,8 +1563,7 @@ class conv_tiler_impl_t {
             case tiler_mode_t::lookup: {
                 const auto params = const_conv_lookup_table().find(cfg.key());
                 if (!params.is_empty() && chk.is_ok(params.blocking())) {
-                    gpu_info() << "[INFO] Using lookup table config: "
-                               << params.str();
+                    gpu_info() << "Using lookup table config: " << params.str();
                     params_gen_ = params_generator_t(tune_level, simd_size, chk,
                             level_tile_sets, params);
                 } else {
diff --git a/src/gpu/intel/jit/ir/send_plan.cpp b/src/gpu/intel/jit/ir/send_plan.cpp
index 6f7c8a6d399..e4bf7c79adc 100644
--- a/src/gpu/intel/jit/ir/send_plan.cpp
+++ b/src/gpu/intel/jit/ir/send_plan.cpp
@@ -579,7 +579,7 @@ std::vector<T> slice(const std::vector<T> &v, int start, int stop) {
 }
 
 const char *fail_2d_header() {
-    return "INFO: can't use 2D send. ";
+    return "Cannot use 2D send. ";
 }
 
 template <typename T0>
diff --git a/src/gpu/intel/jit/pooling/gen_pooling.cpp b/src/gpu/intel/jit/pooling/gen_pooling.cpp
index 66ce3de1d2a..1ee738650d8 100644
--- a/src/gpu/intel/jit/pooling/gen_pooling.cpp
+++ b/src/gpu/intel/jit/pooling/gen_pooling.cpp
@@ -136,8 +136,6 @@ status_t gen_pooling_fwd_t::init(impl::engine_t *engine) {
         gpu_assert(!t.needs_zero_out);
 
         if (t.arg_key == DNNL_ARG_UNDEF) {
-            gpu_assert(!t.needs_reorder);
-            gpu_assert(!t.needs_zero_out);
             gpu_error_not_expected();
             continue;
         }
diff --git a/src/gpu/intel/jit/reorder/gen_reorder.cpp b/src/gpu/intel/jit/reorder/gen_reorder.cpp
index 3b354765b3e..8e60998d830 100644
--- a/src/gpu/intel/jit/reorder/gen_reorder.cpp
+++ b/src/gpu/intel/jit/reorder/gen_reorder.cpp
@@ -197,8 +197,6 @@ status_t gen_reorder_t::pd_t::init_kernel_info() {
         gpu_assert(!t.needs_zero_out);
 
         if (t.arg_key == DNNL_ARG_UNDEF) {
-            gpu_assert(!t.needs_reorder);
-            gpu_assert(!t.needs_zero_out);
             gpu_error_not_expected();
             continue;
         }
diff --git a/src/gpu/intel/jit/utils/trace.cpp b/src/gpu/intel/jit/utils/trace.cpp
index cd41a0acbbe..ca99bf5486e 100644
--- a/src/gpu/intel/jit/utils/trace.cpp
+++ b/src/gpu/intel/jit/utils/trace.cpp
@@ -36,8 +36,7 @@ ir_utils::debug_profiler_t &get_trace_profiler() {
 void trace_pass(
         const char *pass_name, const stmt_t &stmt, const ir_context_t &ir_ctx) {
     trace_stop(pass_name);
-    gpu_trace() << "=== After " << pass_name;
-    gpu_trace() << stmt;
+    gpu_trace() << "=== After " << pass_name << "\n" << stmt;
     auto grf_usage = get_grf_usage(stmt, ir_ctx.hw().grf_size());
     if (!grf_usage.is_empty()) gpu_trace() << grf_usage;
     gpu_trace() << ir_ctx.cset();