search for: pr17769

...c instead > for assembling. > > I'm not very familiar with the code around the asm parser, so I expect > more detailed comments from others to follow. > > Cheers, > Amara > > -------------- next part -------------- A non-text attachment was scrubbed... Name: 0001-PR17769-Initial-implementation-of-ldr-pseudo.patch Type: application/octet-stream Size: 12883 bytes Desc: not available URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20131112/2200e8ac/attachment.obj>

Hi David, Thanks for your efforts here. I have a few comments on your patch, although I realise it's still a work in progress. +class ConstantPool { + MCSymbol *Label; + typedef std::vector<const MCExpr*> EntryVecTy; Use a SmallVector here? + MCSymbol *getLabel() {return Label;} + size_t getNumEntries() {return Entries.size();} + const MCExpr *getEntry(size_t Num) {return

...ev/2013-October/066808.html > [2]: http://llvm.org/bugs/show_bug.cgi?id=17769 > > -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, > hosted by The Linux Foundation > > -------------- next part -------------- A non-text attachment was scrubbed... Name: 0001-PR17769-Initial-implementation-of-ldr-pseudo.patch Type: application/octet-stream Size: 11398 bytes Desc: not available URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20131111/b0ffc376/attachment.obj>

In an earlier email[1] I proposed adding support for the ldr pseud-instruction to the ARM integrated assembler. After some discussion the overall consensus seemed to be that it was worth adding. One concern was that we needed to have adequate testing. I promised to provide more details on what the behavior should be and provide some tests before starting the implementation. The FileCheck-ified