[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