[PATCH] D52010: RegAllocFast: Rewrite and improve

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 1 12:30:28 PDT 2018


arsenm added a comment.

In https://reviews.llvm.org/D52010#1273686, @MatzeB wrote:

> > I'm now depending on this for a change I'm working on rather than figuring out how to fix kill flag handling
>
> I'm not sure what exactly you mean by that. But just as a warning: This patch will lead to more kill flags being added than before, but it does not guarantee that every last use has a kill flag (especially vregs live across blocks will still lack kill flags).


This patch avoids a problem I'm having applying my partial allocation patch to RegAllocFast. It seems to have a bug now where the first time a physical register is used, it immediately sets kill on it even though there are further uses of the same def


Repository:
  rL LLVM

https://reviews.llvm.org/D52010





More information about the llvm-commits mailing list