[PATCH] A bit-tracking DCE Pass

hfinkel at anl.gov hfinkel at anl.gov
Sun Feb 15 18:39:41 PST 2015


In http://reviews.llvm.org/D7531#123925, @hfinkel wrote:

> In http://reviews.llvm.org/D7531#123922, @jmolloy wrote:
>
> > Hi Hal,
> >
> > I'm looking at using very similar information in a pass I'm writing (live bits information). Would it be easy enough to move the AliveBits stuff into a utility class so it can be called from elsewhere?
>
>
> Yes, I think we could separate the analysis part from the part that removes the completely-dead parts. If you can tell me, what are you working on?


Upon further reflection, I'm not sure what the right refactoring for this would be. It could be an analysis pass, but it could also be a utility class exported from here. An actual analysis pass may not be particularly useful here because I don't think it would be likely to be preserved by any non-trivial transformation. I don't think there is any real way to "locally repair" the analysis after making a change (you could update it to get  conservative answer, but because you never remove live bits under iteration, you'd not get the same answer as you might running the analysis from the beginning).

As a result, we might want to get this functionality in-tree, and then refactor it there.

> 

> 

> > Cheers,

> 

> > 

> 

> > James

> 



http://reviews.llvm.org/D7531

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list