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

Philip Reames listmail at philipreames.com
Mon Jul 21 10:08:26 PDT 2014


================
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?

http://reviews.llvm.org/D4574






More information about the llvm-commits mailing list