[all-commits] [llvm/llvm-project] 20a568: [Propeller]: Use a descriptive temporary symbol na...

[Rahman Lavaee via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 20a568c29db0117a6f32861826fedbb33329e759
      https://github.com/llvm/llvm-project/commit/20a568c29db0117a6f32861826fedbb33329e759
  Author: Rahman Lavaee <rahmanl at google.com>
  Date:   2020-08-05 (Wed, 05 Aug 2020)

  Changed paths:
    M llvm/include/llvm/CodeGen/MachineBasicBlock.h
    M llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp
    M llvm/lib/CodeGen/MachineBasicBlock.cpp
    M llvm/test/CodeGen/X86/basic-block-sections-clusters.ll
    M llvm/test/CodeGen/X86/basic-block-sections-listbb.ll
    M llvm/test/CodeGen/X86/basic-block-sections.ll
    M llvm/test/DebugInfo/X86/basic-block-sections_1.ll

  Log Message:
  -----------
  [Propeller]: Use a descriptive temporary symbol name for the end of the basic block.

This patch changes the functionality of AsmPrinter to name the basic block end labels as LBB_END${i}_${j}, with ${i} being the identifier for the function and ${j} being the identifier for the basic block. The new naming scheme is consistent with how basic block labels are named (.LBB${i}_{j}), and how function end symbol are named (.Lfunc_end${i}) and helps to write stronger tests for the upcoming patch for BB-Info section (as proposed in https://lists.llvm.org/pipermail/llvm-dev/2020-July/143512.html). The end label is used with basicblock-labels (BB-Info section in future) and basicblock-sections to compute the size of basic blocks and basic block sections, respectively. For BB sections, the section containing the entry basic block will not have a BB end label since it already gets the function end-label.
This label is cached for every basic block (CachedEndMCSymbol) like the label for the basic block (CachedMCSymbol).

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