[llvm-commits] [llvm] r104057 - in /llvm/trunk: include/llvm/MC/MCStreamer.h lib/MC/MCAsmStreamer.cpp lib/MC/MCNullStreamer.cpp lib/MC/MCParser/AsmParser.cpp

[Eric Christopher]

On May 18, 2010, at 3:11 PM, Chris Lattner wrote:

> 
> On May 18, 2010, at 2:16 PM, Eric Christopher wrote:
> 
>> Author: echristo
>> Date: Tue May 18 16:16:04 2010
>> New Revision: 104057
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=104057&view=rev
>> Log:
>> Make EmitTBSSSymbol take a section argument so that we can find it later.
>> Fix up callers and users.
> 
> Should the syntax in the .s file also allow this to be specified (like .zerofill)?  If not, it seems strange for the EmitTBSSSymbol hook to take a section.

I agree, but I needed it for the object file out bits since as far as I can tell I don't have access to the named sections in other places.  Maybe a new type of symbol data instead?

And no, it probably shouldn't be specified, it's just a shortcut like .bss or .comm.

-eric