Page MenuHomePhabricator

Use Propeller ID instead of MBB IDs.
Needs ReviewPublic

Authored by rahmanl on Mon, Apr 19, 7:26 PM.

Details

Summary

Let Propeller use specialized IDs for basic blocks, instead of MBB number.

This allows optimizations not just prior to asm-printer, but throughout the entire codegen.

Background

Today Propeller uses machine basic block (MBB) IDs, which already exist, to map native assembly to machine IR. This is done as follows.

  • These IDs are captured and dumped into a specially created section named “bb_addr_map” and is created just before the AsmPrinter pass which writes out object files. This also ensures that we have a mapping that is close to assembly.
  • Annotation works by taking a virtual address of an instruction and looking up the bb_addr_map to find the MBB ID it corresponds to.
  • While this works well today, we need to do better when we scale Propeller to target other Machine IR optimizations like spill code optimization. Register allocation happens earlier in the Machine IR pipeline and we need an annotation mechanism that is valid at that point.
  • The current scheme will not work in this scenario because the MBB ID of a particular basic block is not fixed and changes over the course of codegen (via renumbering, adding, and removing the basic blocks).
  • In other words, the MBB IDs do not provide a one-to-one correspondence throughout the lifetime of Machine IR, due to their volatility. Profile annotation using MBB IDs is restricted to be fixed point; only valid at the exact point where it was dumped.
  • Further, the object code can only be dumped before AsmPrinter and cannot be dumped at an arbitrary point in the Machine IR pass pipeline. Hence, MBB IDs are not suitable and we need something else.
Solution

We propose using unique incremental Propeller IDs for basic blocks instead of MBB IDs. These IDs are assigned upon the creation of machine basic blocks. We modify MachineFunction::CreateMachineBasicBlock to assign the Propeller ID to every newly created basic block. It assigns MachineFunction::NextPropellerID to the Propeller ID and then increments it, which ensures having unique IDs.

To ensure correct profile attribution, multiple equivalent compilations must generate the same Propeller IDs. This is guaranteed as long as the MachineFunction passes run in the same order. Since the NextPropellerID variable is scoped to MachineFunction, interleaving of codegen for different functions won't cause any inconistencies.

Impact on Size of the llvm_bb_addr_map Section

Emitting the Propeller ID results in a 23% increase in the size of the llvm_bb_addr_map section for the clang binary.

Diff Detail

Unit TestsFailed

TimeTest
340 msx64 debian > LLVM-Unit.CodeGen/_/CodeGenTests::LexicalScopesTest.BlockScopes
Note: Google Test filter = LexicalScopesTest.BlockScopes [==========] Running 1 test from 1 test case. [----------] Global test environment set-up.
20 msx64 debian > LLVM-Unit.CodeGen/_/CodeGenTests::LexicalScopesTest.FlatLayout
Note: Google Test filter = LexicalScopesTest.FlatLayout [==========] Running 1 test from 1 test case. [----------] Global test environment set-up.
20 msx64 debian > LLVM-Unit.CodeGen/_/CodeGenTests::LexicalScopesTest.FuncWithEmptyGap
Note: Google Test filter = LexicalScopesTest.FuncWithEmptyGap [==========] Running 1 test from 1 test case. [----------] Global test environment set-up.
20 msx64 debian > LLVM-Unit.CodeGen/_/CodeGenTests::LexicalScopesTest.FuncWithRealGap
Note: Google Test filter = LexicalScopesTest.FuncWithRealGap [==========] Running 1 test from 1 test case. [----------] Global test environment set-up.
40 msx64 debian > LLVM-Unit.CodeGen/_/CodeGenTests::LexicalScopesTest.InlinedScopes
Note: Google Test filter = LexicalScopesTest.InlinedScopes [==========] Running 1 test from 1 test case. [----------] Global test environment set-up.
View Full Test Results (14 Failed)

Event Timeline

rahmanl created this revision.Mon, Apr 19, 7:26 PM
rahmanl updated this revision to Diff 338689.Mon, Apr 19, 7:38 PM

Refactoring.

rahmanl updated this revision to Diff 341389.Wed, Apr 28, 7:51 PM
  • Fix tests.
rahmanl edited the summary of this revision. (Show Details)Thu, Apr 29, 8:02 PM
rahmanl updated this revision to Diff 343803.Fri, May 7, 8:06 PM
  • Fix tests.
  • Add support for MIR printing and parsing.
rahmanl published this revision for review.Fri, May 7, 8:08 PM
Herald added a project: Restricted Project. · View Herald TranscriptFri, May 7, 8:08 PM
rahmanl edited the summary of this revision. (Show Details)Fri, May 7, 8:09 PM