[llvm-commits] [PATCH] [MC] .pushsection / .popsection support

Joerg Sonnenberger joerg at britannica.bec.de
Tue Feb 15 16:21:39 PST 2011


On Tue, Feb 15, 2011 at 11:16:22PM +0100, Joerg Sonnenberger wrote:
> - Removing CurSection / PrevSection adds a lot of complications for no
>   gain. There is no constructor of MCStreamer to set up the initial
>   element, so all functions would have to deal with the section stack
>   being empty. Push would would have to insert an element just after the
>   top, etc., so it doesn't even simplify anything.

Scratch the part about the constructor. Attached is the alternative
version removing CurSection / PrevSection. I still believe this is less
clean than the other patch, but be my guest for either patch.

Joerg
-------------- next part --------------
A non-text attachment was scrubbed...
Name: pushpop2.diff
Type: text/x-diff
Size: 22074 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20110216/edc68046/attachment.diff>


More information about the llvm-commits mailing list