diff options
| author | Fangrui Song <maskray@google.com> | 2020-07-22 13:10:59 -0700 |
|---|---|---|
| committer | Fangrui Song <maskray@google.com> | 2020-07-22 13:12:19 -0700 |
| commit | 5724c8ba29baef986a4a9ee77df98f638ee9e641 (patch) | |
| tree | f15409780041f212a9c9b8371ee8d559ed4ac1ac /clang/lib/Sema/DeclSpec.cpp | |
| parent | fdb45f54b6c0de00ba3bfa5dd157161ebb12799d (diff) | |
| download | llvm-5724c8ba29baef986a4a9ee77df98f638ee9e641.zip llvm-5724c8ba29baef986a4a9ee77df98f638ee9e641.tar.gz llvm-5724c8ba29baef986a4a9ee77df98f638ee9e641.tar.bz2 | |
Temporarily revert D83903 "[PGO] Enable the extended value profile buckets for mem op sizes."
`__llvm_profile_instrument_memop` transitively calls calloc, thus calloc
should not be instrumented.
I saw a
`calloc -> __llvm_profile_instrument_memop -> calloc -> __llvm_profile_instrument_memop -> ...`
infinite loop leading to stack overflow
when the malloc implementation (e.g. tcmalloc) is built and instrumented along with the application.
We should figure out the library calls which may be instrumented and disable
their instrumentation before rolling out this change.
Reviewed By: yamauchi
Differential Revision: https://reviews.llvm.org/D84358
Diffstat (limited to 'clang/lib/Sema/DeclSpec.cpp')
0 files changed, 0 insertions, 0 deletions