[llvm-commits] patch: partial DSE

[Dan Gohman]

On Oct 31, 2011, at 5:38 PM, Peter Cooper wrote:

> Hi
> 
> Please review this patch to allow DSE to trim stores as opposed to deleting them.

Looks ok to me. Be careful with the word "partial" here, because that
word has a specific meaning in the context of DSE.

One misc comment; please put this enum in an anonymous namespace, to
avoid namespace polution:

> +enum OverwriteResult
> +{
> +  OverwriteComplete,
> +  OverwriteEnd,
> +  OverwriteUnknown
> +};

Dan