[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