[PATCH] D33114: [AMDGPU] Turn on the new waitcnt insertion pass. Adjust tests.

Bas Nieuwenhuizen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 16 17:05:51 PDT 2017


bnieuwenhuizen added a comment.

In https://reviews.llvm.org/D33114#756272, @kanarayan wrote:

> Any update on the radv testing/timeline? Assuming all is well, I will check in this change today.


We still got a regression with radv, using the Sascha Willems distancefieldfonts demo from

https://github.com/SaschaWillems/Vulkan

image showing issue: https://people.freedesktop.org/~bnieuwenhuizen/radv_regression.png

full log regressed: https://people.freedesktop.org/~bnieuwenhuizen/waitcnt_regression.log
full log correct: https://people.freedesktop.org/~bnieuwenhuizen/waitcnt_correct.log

shader that probably gets miscompiled: https://people.freedesktop.org/~bnieuwenhuizen/waitcnt_shader.llvm


https://reviews.llvm.org/D33114





More information about the llvm-commits mailing list