<html>
<head>
<base href="https://bugs.freedesktop.org/">
</head>
<body><table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>Bug ID</th>
<td><a class="bz_bug_link
bz_status_NEW "
title="NEW - test_vec4_cmod_propagation.cpp:372:26: error: ‘brw_reg’ is an inaccessible base of ‘brw::dst_reg’"
href="https://bugs.freedesktop.org/show_bug.cgi?id=98485">98485</a>
</td>
</tr>
<tr>
<th>Summary</th>
<td>test_vec4_cmod_propagation.cpp:372:26: error: ‘brw_reg’ is an inaccessible base of ‘brw::dst_reg’
</td>
</tr>
<tr>
<th>Product</th>
<td>Mesa
</td>
</tr>
<tr>
<th>Version</th>
<td>13.0
</td>
</tr>
<tr>
<th>Hardware</th>
<td>x86-64 (AMD64)
</td>
</tr>
<tr>
<th>OS</th>
<td>Linux (All)
</td>
</tr>
<tr>
<th>Status</th>
<td>NEW
</td>
</tr>
<tr>
<th>Keywords</th>
<td>bisected, regression
</td>
</tr>
<tr>
<th>Severity</th>
<td>normal
</td>
</tr>
<tr>
<th>Priority</th>
<td>medium
</td>
</tr>
<tr>
<th>Component</th>
<td>Mesa core
</td>
</tr>
<tr>
<th>Assignee</th>
<td>mesa-dev@lists.freedesktop.org
</td>
</tr>
<tr>
<th>Reporter</th>
<td>vlee@freedesktop.org
</td>
</tr>
<tr>
<th>QA Contact</th>
<td>mesa-dev@lists.freedesktop.org
</td>
</tr>
<tr>
<th>CC</th>
<td>currojerez@riseup.net, itoral@igalia.com
</td>
</tr></table>
<p>
<div>
<pre>mesa: e6aeeace6953a7007d98082e3f44bff40a44106d (master 13.1.0-devel)
CXX test_vec4_cmod_propagation.o
test_vec4_cmod_propagation.cpp: In member function ‘virtual void
cmod_propagation_test_intervening_dest_write_Test::TestBody()’:
test_vec4_cmod_propagation.cpp:372:26: error: ‘brw_reg’ is an inaccessible base
of ‘brw::dst_reg’
bld.ADD(offset(dest, 2), src0, src1);
^
66fcfa6894ab61a8cb70955f4a4113729e4a8099 is the first bad commit
commit 66fcfa6894ab61a8cb70955f4a4113729e4a8099
Author: Iago Toral Quiroga <<a href="mailto:itoral@igalia.com">itoral@igalia.com</a>>
Date: Mon Oct 3 13:33:12 2016 +0200
i965/vec4: make offset() work in terms of a simd width and scalar
components
So that it has the same semantics as the scalar backend implementation. The
helper will now take a simd width (which is always 8 in vec4 mode) and step
as many scalar components as specified by that width, respecting the size
of
the scalar channels.
v2 (Curro):
- Remove the assertion in offset(), byte_offset() has the same checks.
- Use byte_offset() directly instead of add_byte_offset().
- Make things more clear by explicitly including the vertical stride
in the byte offset expression.
Reviewed-by: Francisco Jerez <<a href="mailto:currojerez@riseup.net">currojerez@riseup.net</a>>
:040000 040000 61fda73ca05d5d657615121949183d07f9b05632
2c08f32494029dac5aabfcf483b70c9314d91750 M src
bisect run success</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
<li>You are the QA Contact for the bug.</li>
</ul>
</body>
</html>