[all-commits] [llvm/llvm-project] 8287db: [InstrProf] Report error when merging temporal pro...

Ellis Hoag via All-commits all-commits at lists.llvm.org
Fri Aug 11 08:40:46 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 8287db22beb2ae3b436f80d646802ddcc261d476
      https://github.com/llvm/llvm-project/commit/8287db22beb2ae3b436f80d646802ddcc261d476
  Author: Ellis Hoag <ellis.sparky.hoag at gmail.com>
  Date:   2023-08-11 (Fri, 11 Aug 2023)

  Changed paths:
    M compiler-rt/lib/profile/InstrProfilingMerge.c
    A compiler-rt/test/profile/instrprof-merge-error.c

  Log Message:
  -----------
  [InstrProf] Report error when merging temporal profiles

Temporal profiles do not support profile merging at runtime. The reason
is that each function can only store one timestamp and the
`llmv-profdata` tool is responsible for sampling traces during the merge
command. Report an error in `__llvm_profile_merge_from_buffer()` if
merging a temporal profile.

The added test also checks that lightweight profiles throw an error,
although this may be fixed in https://reviews.llvm.org/D157632

Reviewed By: MaskRay, zequanwu

Differential Revision: https://reviews.llvm.org/D157664




More information about the All-commits mailing list