[PATCH] D24684: AMDGPU: Remove scheduling info from si_mask_branch

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 6 09:38:07 PDT 2016


arsenm added a comment.

In https://reviews.llvm.org/D24684#545999, @nhaehnle wrote:

> Yes, the SchedRW part I agree with, but why the hasNoSchedulingInfo? Is there some additional interaction? FWIW, the change itself looks fine to me apart from that.


I thought this was supposed to produce an error when the scheduling model is supposed to be complete. I'm not sure why this isn't actually erroring without it


https://reviews.llvm.org/D24684





More information about the llvm-commits mailing list