[PATCH] D66936: [DFAPacketizer] Track resources for packetized instructions

[James Molloy via llvm-commits]

No worries, thanks for taking affirmative action!

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.

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).

James

On Mon, 9 Sep 2019 at 13:40, Simon Pilgrim via Phabricator <
reviews at reviews.llvm.org> wrote:

> RKSimon added a comment.
>
> In D66936#1662832 <https://reviews.llvm.org/D66936#1662832>, @RKSimon
> wrote:
>
> > @jmolloy This is causing "compiler is out of heap space" errors on my
> VS2017 and VS2019 all targets builds:
> >
> >   e:\llvm\ninja17\lib\target\hexagon\hexagongendfapacketizer.inc(4892) :
> fatal error C1002: compiler is out of heap space in pass 2
> >
>
>
> I'm sorry but I had to revert this at rL371393 <
> https://reviews.llvm.org/rL371393>, 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.
>
>
> Repository:
>   rG LLVM Github Monorepo
>
> CHANGES SINCE LAST ACTION
>   https://reviews.llvm.org/D66936/new/
>
> https://reviews.llvm.org/D66936
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190909/700a975b/attachment.html>