Displaying 4 results from an estimated 4 matches for "constantpoolmapty".
2013 Dec 17
0
[LLVMdev] Implementing the ldr pseudo instruction in ARM integrated assembler
Hi David,
Maybe I’m just blind, but where’s the code to handle the .ltorg directive? Is that a separate patch, maybe? Without that, this is not going to be usable in any circumstance using subsections-via-symbols.
+typedef std::map<const MCSection *, ConstantPool> ConstantPoolMapTy;
This feels odd to me. Can you elaborate a bit more on the data structure choices?? I would have expected constants to be grouped together explicitly by section and then a nested loop over those sections instead. As-is, won’t this potentially cause lots of switching back and forth between sections...
2013 Dec 17
2
[LLVMdev] Implementing the ldr pseudo instruction in ARM integrated assembler
...r easy reading). I believe many of your concerns were addressed. See
below for a detailed response.
> Maybe I'm just blind, but where's the code to handle the .ltorg directive?
It is implemented in patch 0003 in this email.
> +typedef std::map<const MCSection *, ConstantPool> ConstantPoolMapTy;
>
> This feels odd to me. Can you elaborate a bit more on the data structure
> choices?? I would have expected constants to be grouped together
> explicitly by section and then a nested loop over those sections instead.
> As-is, won't this potentially cause lots of switching ba...
2013 Nov 16
2
[LLVMdev] Implementing the ldr pseudo instruction in ARM integrated assembler
Moving discussion to llvm-commits now that I have a more developed
implementation:
http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20131111/195401.
html
> -----Original Message-----
> From: llvmdev-bounces at cs.uiuc.edu [mailto:llvmdev-bounces at cs.uiuc.edu] On
> Behalf Of David Peixotto
> Sent: Tuesday, November 12, 2013 11:09 AM
> To: 'Amara Emerson'
>
2013 Dec 17
0
[LLVMdev] Implementing the ldr pseudo instruction in ARM integrated assembler
...estions below, and I’ll follow up on the new patch separately.
>> Maybe I'm just blind, but where's the code to handle the .ltorg directive?
>
> It is implemented in patch 0003 in this email.
Excellent.
>
>> +typedef std::map<const MCSection *, ConstantPool> ConstantPoolMapTy;
>>
>> This feels odd to me. Can you elaborate a bit more on the data structure
>> choices?? I would have expected constants to be grouped together
>> explicitly by section and then a nested loop over those sections instead.
>> As-is, won't this potentially cause...