<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#c50">Comment # 50</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:jason@jlekstrand.net" title="Jason Ekstrand <jason@jlekstrand.net>"> <span class="fn">Jason Ekstrand</span></a>
</span></b>
<pre>(In reply to Iago Toral from <a href="show_bug.cgi?id=84566#c49">comment #49</a>)
<span class="quote">> 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.</span >
Go ahead and remove it.</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>