[PATCH] D39743: [ValueLattice] Add CompactValueLatticeElement.

[Florian Hahn via Phabricator via llvm-commits]

fhahn added a comment.

In https://reviews.llvm.org/D39743#918748, @reames wrote:

>


> Can I ask why you chose to tackle this one?  What motivated you to care about the memory consumption of LVI?  Just curious about the background and motivating test cases.

There is no particular test case I am looking at. But I want to use ValueLatticeElement for values in IPSCCP (not just parameters, as it is at the moment), and thought it would be a good idea to try to get a more compact representation before that. Also, the more compact representation should make it easier to add additional lattices in the future.


https://reviews.llvm.org/D39743