[PATCH] D87120: [HeapProf] Heap profiling runtime support

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 4 16:26:49 PDT 2020


MaskRay added a comment.

I pushed a5d6af421d625c78bfb0f63830b51863ff0f0877 <https://reviews.llvm.org/rGa5d6af421d625c78bfb0f63830b51863ff0f0877> to remove annoying clang-tidy readability-identifier-naming warnings. There is a minor compilation error related to stdlib.h. After fixing it, many tests fail because `-fmemprof` was recently renamed to `-fmemory-profile`. Can you upload a new diff?



================
Comment at: compiler-rt/lib/heapprof/heapprof_allocator.cpp:26
+#include <sched.h>
+#include <stdio.h>
+#undef errno
----------------
include stdlib.h 

I get: `error: use of undeclared identifier 'malloc'`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87120



More information about the llvm-commits mailing list