[PATCH] D41292: [AMDGPU] Fixed incorrect uniform branch condition

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 15 09:57:24 PST 2017


arsenm added a comment.

In https://reviews.llvm.org/D41292#956918, @tpr wrote:

> Ah, I didn't see r286134.
>
> How do you mean, partially revert it? It looks like I want to completely revert it and fix the comment for semantic correctness, then think about optimizing the cases when the s_and is not needed.


I mean mostly keep it, just re-insert the and when the branch condition isn't known to be a compare like it implicitly assumes


https://reviews.llvm.org/D41292





More information about the llvm-commits mailing list