search for: emitdebuglabel

I thought of a simpler way to do this which is no more of a hack than the way labels are noramally output. Create an EmitDebugLabel and then in MCAsminfo, add another variable for the suffix for debug labels which can by default be just ":". Then for Mips I can change it to "=." On 11/03/2012 01:40 PM, Reed Kotler wrote: > I'm working on a patch for this problem for mips16; unfortunately it's...

...I'm not sure the best nomenclature for this. In the gcc mips16 patch they call "xxx=." a debug label. The other terminology they use is byte pointer vs ISA-encoded address (ISA-encoded meaning this one bit in the case of mips16 ISA). I'm planning to add a virtual method called EmitDebugLabel to MCStreamer which just calls method EmitLabel. Because of some luck, for exception handling at least, I really only need to EmitDebugLabel for eh_func_beginXX. The other entries in the exception table are all a-b, except for eh_func_beginXX, the 1 bits will cancel each other out. I don't...

mips16 has some unusual requirements for label symbols in .s. I think that arm thumb has the same issue but have not located yet how this is handled. When you have a label of an instruction in mips16, when you reference the label, the linker adds a 1 to the address. When you call an odd numbered address, the procesor switches to mips16 mode and when you call an even numbered address it

...e check and ran the test-suite. On 12/15/2012 08:23 PM, NAKAMURA Takumi wrote: > Author: chapuni > Date: Sat Dec 15 22:23:20 2012 > New Revision: 170280 > > URL: http://llvm.org/viewvc/llvm-project?rev=170280&view=rev > Log: > MCPureStreamer.cpp: Try to fix build, pruning EmitDebugLabel(). > > Modified: > llvm/trunk/lib/MC/MCPureStreamer.cpp > > Modified: llvm/trunk/lib/MC/MCPureStreamer.cpp > URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/MC/MCPureStreamer.cpp?rev=170280&r1=170279&r2=170280&view=diff > ===============================...

...ot fail for me when I tested it. On 12/15/2012 08:23 PM, NAKAMURA Takumi wrote: > Author: chapuni > Date: Sat Dec 15 22:23:20 2012 > New Revision: 170280 > > URL: http://llvm.org/viewvc/llvm-project?rev=170280&view=rev > Log: > MCPureStreamer.cpp: Try to fix build, pruning EmitDebugLabel(). > > Modified: > llvm/trunk/lib/MC/MCPureStreamer.cpp > > Modified: llvm/trunk/lib/MC/MCPureStreamer.cpp > URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/MC/MCPureStreamer.cpp?rev=170280&r1=170279&r2=170280&view=diff > ===============================...

...> > Your caution is appreciated, but I think you've done plenty at that > point. =] I would commit it, and if you're worried, get on IRC to > watch the various build bots in case of some failure. I've checked in the code. Basically I'm just adding a new method called EmitDebugLabel which for all ports except for Mips, should be identical to EmitLabel. Because this is in McElfStreamer, it effects all the classes derived from that. I've tried to be careful and the code was reviewed. If for some port I have messed up , it would be better for the person to just fix the c...

...flavors with this patch and no problems arose. On 11/04/2012 11:07 AM, Reed Kotler wrote: > The main idea is to distinguish between emitting normal labels and debug > labels. This is the nomenclature chosen by the gcc people working on > this same problem. > > I have just added one EmitDebugLabel call, because that is all that is > needed for now to get exceptions to work. > > For the general dwarf, the call needs to be made most likely for all > labels apart from the landing pad label. > > By default normal and debug labels will be the same. > > For Mips, I emit deb...

On Sat, Dec 15, 2012 at 12:33 PM, Reed Kotler <rkotler at mips.com> wrote: > I have an approved target independent putback and i've run all that we > have at Mips as well as on x86 " make TEST=simple" > > Is there anything else that is easy to run that I can do before doing the > commit? > > The patch touches a number of basic classes so I'm just

I have an approved target independent putback and i've run all that we have at Mips as well as on x86 " make TEST=simple" Is there anything else that is easy to run that I can do before doing the commit? The patch touches a number of basic classes so I'm just trying to err on the side of caution.