<div dir="ltr">I've reapplied this as rL371399; please do revert (or email and I will revert) if it breaks you again. I'll be keeping an eye on the buildbots too.<div><br></div><div>Cheers,</div><div><br></div><div>James</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 9 Sep 2019 at 13:49, James Molloy <<a href="mailto:jmolloy@google.com">jmolloy@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">No worries, thanks for taking affirmative action!<div><br></div><div>I have a candidate patch that reduces the compile time on GCC and Clang by a large factor. I'm expecting/hoping that the compiler resource usage on VS would also decrease a commensurate amount.</div><div><br></div><div>Would you prefer to patch it in or have me roll forward and you roll back again if the same issue occurs? (do we have a buildbot that fails? I checked my bot mail afterwards but couldn't see anything useful).</div><div><br></div><div>James</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 9 Sep 2019 at 13:40, Simon Pilgrim via Phabricator <<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">RKSimon added a comment.<br>
<br>
In D66936#1662832 <<a href="https://reviews.llvm.org/D66936#1662832" rel="noreferrer" target="_blank">https://reviews.llvm.org/D66936#1662832</a>>, @RKSimon wrote:<br>
<br>
> @jmolloy This is causing "compiler is out of heap space" errors on my VS2017 and VS2019 all targets builds:<br>
><br>
>   e:\llvm\ninja17\lib\target\hexagon\hexagongendfapacketizer.inc(4892) : fatal error C1002: compiler is out of heap space in pass 2<br>
><br>
<br>
<br>
I'm sorry but I had to revert this at rL371393 <<a href="https://reviews.llvm.org/rL371393" rel="noreferrer" target="_blank">https://reviews.llvm.org/rL371393</a>>, I didn't get very far in triaging what is the causing the issue but if you have a candidate patch I'm happy to test it.<br>
<br>
<br>
Repository:<br>
  rG LLVM Github Monorepo<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D66936/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D66936/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D66936" rel="noreferrer" target="_blank">https://reviews.llvm.org/D66936</a><br>
<br>
<br>
<br>
</blockquote></div>
</blockquote></div>