<html>
<head>
<base href="https://bugs.freedesktop.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - Unify the format conversion code"
href="https://bugs.freedesktop.org/show_bug.cgi?id=84566#c49">Comment # 49</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - Unify the format conversion code"
href="https://bugs.freedesktop.org/show_bug.cgi?id=84566">bug 84566</a>
from <span class="vcard"><a class="email" href="mailto:itoral@igalia.com" title="Iago Toral <itoral@igalia.com>"> <span class="fn">Iago Toral</span></a>
</span></b>
<pre>Jason you had this commit in your original branch: "MAYBEREVERT: Fill X
components with 1"
That basically makes packing to padded formats (like RGBX) set X=1. In the
commit message you mention that you are not sure you like this... my opinion is
that this should not be necessary and it degrades performance when packing to
these format, so I would be more for removing this one. What do you think?
FWIW, I have checked that removing this commit does not affect piglit results.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the QA Contact for the bug.</li>
</ul>
</body>
</html>