[all-commits] [llvm/llvm-project] f52e61: [BOLT][DWARF] Replace MD5 with hash_combine

Alexander Yermolovich via All-commits all-commits at lists.llvm.org
Thu Jul 20 11:51:38 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: f52e61f3d37bf105f99867dd939cb46caee63ffe
      https://github.com/llvm/llvm-project/commit/f52e61f3d37bf105f99867dd939cb46caee63ffe
  Author: Alexander Yermolovich <ayermolo at meta.com>
  Date:   2023-07-20 (Thu, 20 Jul 2023)

  Changed paths:
    M bolt/lib/Core/DIEBuilder.cpp

  Log Message:
  -----------
  [BOLT][DWARF] Replace MD5 with hash_combine

Slight performance improvement, based on perf.

Collected on clang-17 built with DWARF4 + split dwarf.
MD5
8:46.50 real,   713.38 user,    64.19 sys,      0 amem, 41933136 mmem
8:27.44 real,   708.55 user,    63.83 sys,      0 amem, 41906576 mmem
8:40.37 real,   724.63 user,    62.56 sys,      0 amem, 42319572 mmem

hash_combine

8:03.99 real,   681.92 user,    60.04 sys,      0 amem, 42459204 mmem
8:02.92 real,   685.20 user,    62.56 sys,      0 amem, 41879164 mmem
7:57.85 real,   690.27 user,    60.12 sys,      0 amem, 41806240 mmem

Reviewed By: maksfb

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




More information about the All-commits mailing list