[all-commits] [llvm/llvm-project] ebad67: [SampleFDO] Port MD5 name table support to extbina...

wmi-11 via All-commits all-commits at lists.llvm.org
Mon Mar 30 22:07:43 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: ebad678857a94c32ce7b6931e9c642b32d278b67
      https://github.com/llvm/llvm-project/commit/ebad678857a94c32ce7b6931e9c642b32d278b67
  Author: Wei Mi <wmi at google.com>
  Date:   2020-03-30 (Mon, 30 Mar 2020)

  Changed paths:
    M llvm/include/llvm/ProfileData/SampleProf.h
    M llvm/include/llvm/ProfileData/SampleProfReader.h
    M llvm/include/llvm/ProfileData/SampleProfWriter.h
    M llvm/lib/ProfileData/SampleProf.cpp
    M llvm/lib/ProfileData/SampleProfReader.cpp
    M llvm/lib/ProfileData/SampleProfWriter.cpp
    M llvm/lib/Transforms/IPO/SampleProfile.cpp
    A llvm/test/Transforms/SampleProfile/Inputs/inline.md5extbinary.afdo
    M llvm/test/Transforms/SampleProfile/profile-format.ll
    M llvm/test/tools/llvm-profdata/roundtrip.test
    M llvm/tools/llvm-profdata/llvm-profdata.cpp
    M llvm/unittests/ProfileData/SampleProfTest.cpp

  Log Message:
  -----------
  [SampleFDO] Port MD5 name table support to extbinary format.

Compbinary format uses MD5 to represent strings in name table. That gives smaller profile without the need of compression/decompression when writing/reading the profile. The patch adds the support in extbinary format. It is off by default but user can choose to enable it.

Note the feature of using MD5 in name table can bring very small chance of name conflict leading to profile mismatch. Besides, profile using the feature won't have the profile remapping support.

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




More information about the All-commits mailing list