From 9482d392a1302948619cde5d01a9cdd94bee7ec2 Mon Sep 17 00:00:00 2001 From: Kenneth Graunke Date: Tue, 9 Dec 2025 03:30:34 -0800 Subject: [PATCH] brw: Fix outdated comments about urb->offset units I recently converted urb->offset to be in bytes on Xe2, but neglected to update these comments that still said OWord. Fixes: 9ffae429752 ("brw: Store brw_urb_inst::offset in bytes on Xe2") Reviewed-by: Lionel Landwerlin Reviewed-by: Rohan Garg Part-of: --- src/intel/compiler/brw/brw_lower_logical_sends.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/intel/compiler/brw/brw_lower_logical_sends.cpp b/src/intel/compiler/brw/brw_lower_logical_sends.cpp index 4ff5b6ed27d..a893642e4fb 100644 --- a/src/intel/compiler/brw/brw_lower_logical_sends.cpp +++ b/src/intel/compiler/brw/brw_lower_logical_sends.cpp @@ -100,7 +100,7 @@ lower_urb_read_logical_send_xe2(const brw_builder &bld, brw_urb_inst *urb) bld.MOV(payload, handle); /* The low 24-bits of the URB handle is a byte offset into the URB area. - * Add the (OWord) offset of the write to this value. + * Add the byte offset of the write to this value. */ if (urb->offset) { bld.ADD(payload, payload, brw_imm_ud(urb->offset)); @@ -221,7 +221,7 @@ lower_urb_write_logical_send_xe2(const brw_builder &bld, brw_urb_inst *urb) bld.MOV(payload, handle); /* The low 24-bits of the URB handle is a byte offset into the URB area. - * Add the (OWord) offset of the write to this value. + * Add the byte offset of the write to this value. */ if (urb->offset) { bld.ADD(payload, payload, brw_imm_ud(urb->offset));