[PATCH] D118203: [CSSPGO] Adjust SampleContextFrameVector to be a shorter SmallVector

Tue Jan 25 17:09:06 PST 2022

wenlei accepted this revision.
wenlei added a comment.
This revision is now accepted and ready to land.

In D118203#3271163 <https://reviews.llvm.org/D118203#3271163>, @hoy wrote:

> In D118203#3271143 <https://reviews.llvm.org/D118203#3271143>, @wenlei wrote:
>
>> Does this lengthen process time?
>
> Not necessarily if reallocaton happens frequently.
>
> Before:
>
>   note: After computeSizeForProfiledFunctions
>   note: Tue Jan 25 10:38:10 2022
>   note: VM: 106.28 GB   RSS: 91.25 GB
>   note: After generateProbeBasedProfile
>   note: Tue Jan 25 11:06:02 2022
>   note: VM: 245.19 GB   RSS: 224.30 GB
>
> After:
>
>   note: After computeSizeForProfiledFunctions
>   note: Tue Jan 25 15:31:13 2022
>   note: VM: 105.95 GB   RSS: 91.04 GB
>   note: After generateProbeBasedProfile
>   note: Tue Jan 25 15:58:03 2022
>   note: VM: 234.29 GB   RSS: 215.39 GB

Thanks for checking. Nice improvements!

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118203