[all-commits] [llvm/llvm-project] 6ebf7c: [InstrProf][compiler-rt] Fix counter section align...

Ellis Hoag via All-commits all-commits at lists.llvm.org
Thu Jun 8 17:26:33 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 6ebf7cd7ed1b6502b3b18985f06b213a53381097
      https://github.com/llvm/llvm-project/commit/6ebf7cd7ed1b6502b3b18985f06b213a53381097
  Author: Ellis Hoag <ellis.sparky.hoag at gmail.com>
  Date:   2023-06-08 (Thu, 08 Jun 2023)

  Changed paths:
    M compiler-rt/lib/profile/InstrProfilingBuffer.c
    A compiler-rt/test/profile/Posix/instrprof-shared-entry-coverage.test

  Log Message:
  -----------
  [InstrProf][compiler-rt] Fix counter section alignment issue

I recently discovered that `.profraw` headers are expected to be 8 byte
aligned.
https://github.com/llvm/llvm-project/blob/643ba926c1f618401c86dc37e659df795db2e1a0/llvm/lib/ProfileData/InstrProfReader.cpp#L503-L506

When function entry coverage mode is used, function counters are single
bytes, so it is likely that the size of the counters section is not 8
byte aligned. We can add padding after the counters section to guarantee
this.

Reviewed By: kyulee, gulfem

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




More information about the All-commits mailing list