[PATCH] D33105: [AMDGPU] Turn register pressure estimation into forward tracker
Valery Pykhtin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon May 15 09:53:32 PDT 2017
vpykhtin added a comment.
In https://reviews.llvm.org/D33105#755186, @rampitec wrote:
> In https://reviews.llvm.org/D33105#755182, @vpykhtin wrote:
>
> > This looks a bit as using carkeys to open a bottle. Should we have single instruction RP diff returning function without changing any state?
>
>
> We probably should, but it probably does not belong to this change.
I would not introduce interface in a hope of such usage. This is really counterintuitive. Lets do the RP diff function instead when we need it.
Repository:
rL LLVM
https://reviews.llvm.org/D33105
More information about the llvm-commits
mailing list