[llvm] r314411 - [LVI] Move LVILatticeVal class to separate header file (NFC).

[Florian Hahn via llvm-commits]

On 29/09/2017 16:49, Philip Reames wrote:
> Are we sure that sharing code is a good idea?  These two transforms have
> evolved fairly independently over time and I'm worried that coupling
> their development is not worth the small amount of code sharing.
>
> Can you provide a sketch of your overall plan here?
My plan was to provide the infrastructure to reason about values and ranges in a re-usable way, so we can use it in other passes as well. Both transforms seem to use the same kind of range information and I do not think sharing the code will have a negative impact on any of the two transforms. If we come up with a more efficient representation, both transforms would benefit.

Do you have any specific concerns?

Cheers,
Florian