<div dir="auto">Still unsure about the heuristic here. I believe that for auto-initialization we want to be quite aggressive with these splits (unlike for regular constant stores).<div dir="auto">Perhaps we should do the split in the case all bytes are pattern bytes? (This is probably another use case for `forInit`)</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Feb 8, 2019, 11:36 Alexander Potapenko via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">glider marked 2 inline comments as not done.<br>
glider added a comment.<br>
<br>
Sorry, didn't mean to mark these done.<br>
<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D57898/new/" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D57898/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D57898" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D57898</a><br>
<br>
<br>
<br>
</blockquote></div>