compressonator icon indicating copy to clipboard operation
compressonator copied to clipboard

line 411 is wrong in bc6h_decode.cpp - will produce incorrect output

Open bernief1 opened this issue 4 years ago • 1 comments

https://github.com/GPUOpen-Tools/compressonator/blob/cb281b97eb05e35ec739a462c71524c4b1eefae3/cmp_compressonatorlib/bc6h/bc6h_decode.cpp#L411

bernief1 avatar Aug 12 '21 21:08 bernief1

I found this too. I think it needs to be use a 'or' | like similar r/g calls where it uses 5 bites. Comments says it's 11:4:4:5, so I think on the w:r:g:b the b needs the 'or' to shift the high bit of that 5-bit value.

bc6h_format.by = header.getvalue(61,4) | //5: by[3:0] (header.getvalue(40,1) << 4); // by[4]

alecazam avatar Oct 06 '21 16:10 alecazam