[llvm] r317753 - AMDGPU: Merge BUFFER_LOAD_DWORD_OFFSET into x2, x4

Michel Dänzer via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 10 02:37:09 PST 2017


Hi Marek,


> I had looked at the issue and it seems variable indexing has always been
> somewhat broken in LLVM and this commit uncovered the bug.

That's unfortunate, but doesn't change the situation that tests which
were previously passing are now failing. If we let these tests keep
failing, we might miss further regressions affecting them.

If there's no solution on the horizon, it would be better to revert this
change until it doesn't break these tests.


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer


More information about the llvm-commits mailing list