[PATCH] D20262: [DSE]Split memset when the memset is small enough to be lowered to stores

[Daniel Berlin via llvm-commits]

The first seems plausible, but datalayout is not optional anymore, IIRC, so
the second should be impossible.


On Tue, May 24, 2016 at 1:23 PM, Jun Bum Lim <junbuml at codeaurora.org> wrote:

> junbuml added inline comments.
>
> ================
> Comment at: lib/Transforms/Scalar/DeadStoreElimination.cpp:238
> @@ +237,3 @@
> +
> +  if (MaxIntSize == 0)
> +    MaxIntSize = 1;
> ----------------
> dberlin wrote:
> > How and when can this happen?
> If there is no information about LegalIntWidth extracted from datalayout,
> this could be 0.  If we remove the target datalayout in input IR, this will
> be 0.
>
>
> http://reviews.llvm.org/D20262
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160524/24044d30/attachment.html>