[PATCH] D11822: Resurrect RegisterAsmStreamer

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 7 14:15:38 PDT 2015


arsenm added a comment.

In http://reviews.llvm.org/D11822#219402, @rafael wrote:

> To be clear, the idea is to avoid *creating* the sections, no avoiding
>  printing them, correct?


Yes, although the specific problem is they are printed. We don't have meaningful sections in the text format.

> Ins't all that you need to do is overload (or add an if to) InitSections?


Probably, but that would still require a new custom MCAsmStreamer subclass. Essentially all we want out of the MCStreamer is what MCBasicAsmStreamer has in it. Essentially anything other than printing instructions and labels are going to be incorrect.

In http://reviews.llvm.org/D11822#219403, @rafael wrote:

> For what it is worth, WebAssembly seems to manage to just use nullptr:
>
>   TextSection = nullptr;
>     DataSection = nullptr;
>     BSSSection = nullptr;
>     ReadOnlySection = nullptr;


This was one of the first things I tried and it doesn't work. WebAssembly does current print .text / .data directives.


http://reviews.llvm.org/D11822





More information about the llvm-commits mailing list