[PATCH] Add support for StackMap section for ELF/Linux systems

Eric Christopher echristo at gmail.com
Mon Jul 21 14:52:26 PDT 2014


On Mon, Jul 21, 2014 at 10:08 AM, Philip Reames
<listmail at philipreames.com> wrote:
> ================
> Comment at: lib/MC/MCObjectFileInfo.cpp:590
> @@ -585,1 +589,3 @@
> +                       SectionKind::getReadOnly());
> +
>  }
> ----------------
> Juergen Ributzka wrote:
>> I am not familiar with the ELF sections, but wouldn't Metadata be a better for for this section. I guess it doesn't work with it, because the JIT won't ask the client to allocate it?
>>
>> Lang knows more about this part of the JIT.
> Worth noting:
> 1) The COFF section uses getMetadata
> 2) getMetadata appears to work in practice
>
> I'm happy to make the change, but don't feel like I understand the implications.  Reading through the documentation, I'm unclear on the semantics.  Lang, can you clarify?
>

Philip: I'm not sure what Lang can clarify past my comments a few days
ago. Was anything in there unclear? (Other than my question about
whether or not the JIT will allocate anything with SHF_ALLOC - if it
doesn't we should fix the JIT).

-eric



More information about the llvm-commits mailing list