[PATCH] D100917: [NewPM] Only invalidate modified functions' analyses in CGSCC passes

Arthur Eubanks via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 4 10:18:20 PDT 2021


aeubanks added a comment.

In D100917#2735651 <https://reviews.llvm.org/D100917#2735651>, @nikic wrote:

> An unfortunate side-effect of this change is that NewPM uses even more memory. tramp3d-v4 is up 20% in max-rss (https://llvm-compile-time-tracker.com/compare.php?from=4ef1f90e4d564b872e3598ccef45adb740eb0f0d&to=d14d84af2f5ebb8ae2188ce6884a29a586dc0a40&stat=max-rss)

Hmm that is a concern. I'm not sure how we want to balance memory vs compile times. Any thoughts?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D100917/new/

https://reviews.llvm.org/D100917



More information about the cfe-commits mailing list