diff options
| author | Kenneth Graunke <[email protected]> | 2014-06-09 02:59:21 -0700 |
|---|---|---|
| committer | Kenneth Graunke <[email protected]> | 2014-06-10 16:38:27 -0700 |
| commit | ecc78eab119ac8fa3df380a80bc94975e986523c (patch) | |
| tree | 08268fabd28102b4ff288c1a5bd80c0f7be138d2 /src | |
| parent | fc19c4aaf17a43c88d789ca9b7f7b2fe0c6f27a1 (diff) | |
i965: Don't use the head sentinel as an fs_inst in Gen4 workaround code.
When walking backwards, we want to stop at the head sentinel, which is
where scan_inst->prev->prev == NULL, not scan_inst->prev == NULL.
Fixes random crashes, as well as valgrind errors.
Signed-off-by: Kenneth Graunke <[email protected]>
Reviewed-by: Matt Turner <[email protected]>
Cc: [email protected]
Diffstat (limited to 'src')
| -rw-r--r-- | src/mesa/drivers/dri/i965/brw_fs.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/src/mesa/drivers/dri/i965/brw_fs.cpp index a8ca9bc9ec0..b485e65adcf 100644 --- a/src/mesa/drivers/dri/i965/brw_fs.cpp +++ b/src/mesa/drivers/dri/i965/brw_fs.cpp @@ -2355,7 +2355,7 @@ fs_visitor::insert_gen4_pre_send_dependency_workarounds(fs_inst *inst) * program. */ for (fs_inst *scan_inst = (fs_inst *)inst->prev; - scan_inst != NULL; + !scan_inst->is_head_sentinel(); scan_inst = (fs_inst *)scan_inst->prev) { /* If we hit control flow, assume that there *are* outstanding |