[PATCH] D9009: Value profiling compiler-rt changes

Betul Buyukkurt via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 12 10:12:39 PST 2015


> davidxl added inline comments.
>
> ================
> Comment at: lib/profile/InstrProfiling.c:190
> @@ +189,3 @@
> +    }
> +    I->ValueCounters = (void *)PerSiteCountsHead;
> +    PerSiteCountsHead = (uint8_t *)VDataPtr;
> ----------------
> Release memory of ValueCounters ?

This is a function that's running only atexit, I thought of adding it in
but then thought of it as unnecessary too due to this being one of the
very last actions of the run time during the lifetime of the program.

>
> http://reviews.llvm.org/D9009
>
>
>
>




More information about the llvm-commits mailing list