[PATCH] D73674: Propeller: LLVM support for basic block sections (Base Patch - Part 2)

[Sriraman Tallam via Phabricator via llvm-commits]

tmsriram marked an inline comment as done.
tmsriram added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/MachineBasicBlock.h:147
+  /// Stores the Section type of the basic block with basic block sections.
+  MachineBasicBlockSection SectionType = MBBS_None;
+
----------------
efriedma wrote:
> I'd like to have MIR support for serializing/deserializing the SectionType, so we can test the BBSectionsPrepare pass separately from the passes that run after it and the asmprinter.
* Function attribute is one way I suppose but if the sections need to be created only for a subset of basic blocks then we would need to emit metadata?  

* I thought of emitting a function attribute like "bbsections" with a string value which encodes the basic block ids that require sections.  

Thoughts?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D73674/new/

https://reviews.llvm.org/D73674