[all-commits] [llvm/llvm-project] e28c39: [BOLT] Reduce the number of emitted symbols. NFCI....

[maksfb via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e28c393bd1c1f22f229d156b0b8283746cdac322
      https://github.com/llvm/llvm-project/commit/e28c393bd1c1f22f229d156b0b8283746cdac322
  Author: maksfb <maks at fb.com>
  Date:   2023-11-06 (Mon, 06 Nov 2023)

  Changed paths:
    M bolt/include/bolt/Core/MCPlusBuilder.h
    M bolt/lib/Core/BinaryContext.cpp
    M bolt/lib/Core/BinaryEmitter.cpp
    M bolt/lib/Core/MCPlusBuilder.cpp
    M bolt/lib/Passes/BinaryPasses.cpp

  Log Message:
  -----------
  [BOLT] Reduce the number of emitted symbols. NFCI. (#70175)

We emit a symbol before an instruction for a number of reasons, e.g. for
tracking LocSyms, debug line, or if the instruction has a label
annotation. Currently, we may emit multiple symbols per instruction.

Reuse the same label instead of creating and emitting new ones when
possible. I'm planning to refactor EH labels as well in a separate diff.

Change getLabel() to return a pointer instead of std::optional<> since
an empty label should be treated identically to no label.