AlexIndustrial/mesa
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

intel/fs: Fix signedness of payload_node_count argument of calculate_payload_ranges().

Browse Source 
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/25020>
This commit is contained in:
Francisco Jerez
2022-07-07 01:12:24 -07:00
committed by Jordan Justen
parent abf8111560
commit 2b7419d090
2 changed files with 7 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/intel/compiler/brw_fs.h
+1 -1
View File
@@ -247,7 +247,7 @@ public:
void assign_gs_urb_setup();
bool assign_regs(bool allow_spilling, bool spill_all);
void assign_regs_trivial();
void calculate_payload_ranges(int payload_node_count,
void calculate_payload_ranges(unsigned payload_node_count,
int *payload_last_use_ip) const;
bool split_virtual_grfs();
bool compact_virtual_grfs();
src/intel/compiler/brw_fs_reg_allocate.cpp
+6 -6
View File
@@ -208,13 +208,13 @@ count_to_loop_end(const bblock_t *block)
unreachable("not reached");
}
void fs_visitor::calculate_payload_ranges(int payload_node_count,
void fs_visitor::calculate_payload_ranges(unsigned payload_node_count,
int *payload_last_use_ip) const
{
int loop_depth = 0;
int loop_end_ip = 0;
for (int i = 0; i < payload_node_count; i++)
for (unsigned i = 0; i < payload_node_count; i++)
payload_last_use_ip[i] = -1;
int ip = 0;
@@ -250,23 +250,23 @@ void fs_visitor::calculate_payload_ranges(int payload_node_count,
*/
for (int i = 0; i < inst->sources; i++) {
if (inst->src[i].file == FIXED_GRF) {
int node_nr = inst->src[i].nr;
unsigned node_nr = inst->src[i].nr;
if (node_nr >= payload_node_count)
continue;
for (unsigned j = 0; j < regs_read(inst, i); j++) {
payload_last_use_ip[node_nr + j] = use_ip;
assert(node_nr + j < unsigned(payload_node_count));
assert(node_nr + j < payload_node_count);
}
}
}
if (inst->dst.file == FIXED_GRF) {
int node_nr = inst->dst.nr;
unsigned node_nr = inst->dst.nr;
if (node_nr < payload_node_count) {
for (unsigned j = 0; j < regs_written(inst); j++) {
payload_last_use_ip[node_nr + j] = use_ip;
assert(node_nr + j < unsigned(payload_node_count));
assert(node_nr + j < payload_node_count);
}
}
}