[all-commits] [llvm/llvm-project] 888d0a: [lld-macho][NFC] Remove redundant StringRef constr...

Daniel Bertalan via All-commits all-commits at lists.llvm.org
Thu Jul 21 06:37:57 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 888d0a5ef2590f9f3e4c6820dfacb800db029d97
      https://github.com/llvm/llvm-project/commit/888d0a5ef2590f9f3e4c6820dfacb800db029d97
  Author: Daniel Bertalan <dani at danielbertalan.dev>
  Date:   2022-07-21 (Thu, 21 Jul 2022)

  Changed paths:
    M lld/MachO/InputFiles.cpp

  Log Message:
  -----------
  [lld-macho][NFC] Remove redundant StringRef construction

It's only used in one branch, so we were unnecessarily calculating the
length of many symbol names.

Tiny speedup when linking chromium_framework on my M1 Mac mini:

x before.txt
+ after.txt

    N           Min           Max        Median           Avg        Stddev
x  10     3.9917109        4.0418     4.0318099     4.0203902   0.021459873
+  10      3.944725      4.053988     3.9708955     3.9825602   0.037257609
Difference at 95.0% confidence
	-0.03783 +/- 0.0285663
	-0.940953% +/- 0.710536%
	(Student's t, pooled s = 0.0304028)

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




More information about the All-commits mailing list