On Tue, Jan 7, 2014 at 2:05 PM, Ian Romanick <i...@freedesktop.org> wrote: > On 01/07/2014 12:31 PM, Thomas Sondergaard wrote: >> This small rearrangement avoids MSVC 2013 ICE. Also, this should be a better >> memory access order. > > Can you explain this better? As far as I can tell, this just changes > the order the indices are visited (and has a spurious, incorrect > whitespace change). This shouldn't affect the correctness of the code > at all.
I can't which thing you're asking about (the ICE, or the memory access pattern), but the memory access pattern claim seems correct. He's reversing the loops, so that consecutive iterations of the inner loop operate on contiguous memory. The correctness of the code doesn't ever seem to have been in question. _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev