[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