On Thu, Jul 6, 2017 at 6:36 PM, Matt Arsenault <arse...@gmail.com> wrote:
>
> On Jul 6, 2017, at 18:31, Connor Abbott <cwabbo...@gmail.com> wrote:
>
> After looking into it some more, I think LLVM won't promote allocas to
> registers at all when there are non-constant indices in the mix, and
> fixing it seems kinda involved. I guess a better solution for now
>
>
> AMDGPUPromoteAlloca does this, but it doesn’t happen very often

Could we just bump the heuristic on maximum size there to match
radeonsi? That way we wouldn't need to have these heuristics in
radeonsi and radv. As-is, we're second-guessing the backend.
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to