[all-commits] [llvm/llvm-project] 63d01b: [MemProf] Add missing header to list of installed ...

Enna1 via All-commits all-commits at lists.llvm.org
Wed Jan 24 23:13:42 PST 2024


  Branch: refs/heads/users/Enna1/memprof-fix-buildbot
  Home:   https://github.com/llvm/llvm-project
  Commit: 63d01bcc3f011faa12d282a28bfb216e5309bb87
      https://github.com/llvm/llvm-project/commit/63d01bcc3f011faa12d282a28bfb216e5309bb87
  Author: xumingjie.enna1 <xumingjie.enna1 at bytedance.com>
  Date:   2024-01-25 (Thu, 25 Jan 2024)

  Changed paths:
    M compiler-rt/include/CMakeLists.txt

  Log Message:
  -----------
  [MemProf] Add missing header to list of installed headers.

There were buildbot failures when running memprof tests:
********************
Failed Tests (12):
  MemProfiler-x86_64-linux :: TestCases/interface_test.cpp
  MemProfiler-x86_64-linux :: TestCases/log_path_test.cpp
  MemProfiler-x86_64-linux :: TestCases/memprof_merge_mib.cpp
  MemProfiler-x86_64-linux :: TestCases/memprof_profile_dump.cpp
  MemProfiler-x86_64-linux :: TestCases/profile_reset.cpp
  MemProfiler-x86_64-linux :: TestCases/unaligned_loads_and_stores.cpp
  MemProfiler-x86_64-linux-dynamic :: TestCases/interface_test.cpp
  MemProfiler-x86_64-linux-dynamic :: TestCases/log_path_test.cpp
  MemProfiler-x86_64-linux-dynamic :: TestCases/memprof_merge_mib.cpp
  MemProfiler-x86_64-linux-dynamic :: TestCases/memprof_profile_dump.cpp
  MemProfiler-x86_64-linux-dynamic :: TestCases/profile_reset.cpp
  MemProfiler-x86_64-linux-dynamic :: TestCases/unaligned_loads_and_stores.cpp

See
- https://lab.llvm.org/buildbot/#/builders/258/builds/8852
- https://lab.llvm.org/buildbot/#/builders/258/builds/12876

I suspect the failure is because when build with
-DLLVM_ENABLE_RUNTIMES=compiler-rt -DCOMPILER_RT_BUILD_SANITIZERS=OFF,
the headers sanitizer/allocator_interface.h and
sanitizer/common_interface_defs.h are not be copied to the build tree,
and not installed.
But in the failed memprof tests,
sanitizer/allocator_interface.h or sanitizer/memprof_interface.h is included.

This patch adds sanitizer/allocator_interface.h and sanitizer/memprof_interface.h
to memprof headers if COMPILER_RT_BUILD_SANITIZERS is false.




More information about the All-commits mailing list