<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Jun 18, 2018 at 4:41 PM, Bas Nieuwenhuizen <span dir="ltr"><<a href="mailto:bas@basnieuwenhuizen.nl" target="_blank">bas@basnieuwenhuizen.nl</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Mon, Jun 18, 2018 at 10:39 PM, Marek Olšák <<a href="mailto:maraeo@gmail.com">maraeo@gmail.com</a>> wrote:<br>
> On Mon, Jun 18, 2018 at 12:12 PM, Bas Nieuwenhuizen<br>
> <<a href="mailto:bas@basnieuwenhuizen.nl">bas@basnieuwenhuizen.nl</a>> wrote:<br>
>><br>
>> On Mon, Jun 18, 2018 at 6:08 PM, Marek Olšák <<a href="mailto:maraeo@gmail.com">maraeo@gmail.com</a>> wrote:<br>
>> > Do you need this if you set compressZ = true for stencil?<br>
>><br>
>> Technically not, then you don't hit the path anymore. Anyway even<br>
>> though neither AMDVLK or RADV anymore if we do that, it still bugged<br>
>> me that the index is wrong.<br>
><br>
><br>
> The way I update addrlib is that I copy all files from the internal repo<br>
> into Mesa and then maybe discard some Mesa-specific changes. This change is<br>
> very likely to get overwritten, so I would like to prevent addrlib changes<br>
> that don't have any impact on current Mesa.<br>
<br>
</span>If I want to fix it "upstream", is the addrlib copy in<br>
<a href="https://github.com/GPUOpen-Drivers/pal" rel="noreferrer" target="_blank">https://github.com/GPUOpen-<wbr>Drivers/pal</a> the right place to update?<br></blockquote><div><br></div>No, that one is an imported copy too. If you want to fix this in the original repository, you can send the patch to my amd address.<br></div><div class="gmail_quote"><br></div><div class="gmail_quote">Marek<br></div></div></div>