[PATCH] Enable StackMap Serialization for COFF

Andy Ayers andya at microsoft.com
Wed Jun 24 17:37:16 PDT 2015


I'm not sure about the COFF flags. Not something that needs to be fixed right away though.


================
Comment at: lib/MC/MCObjectFileInfo.cpp:740
@@ -736,1 +739,3 @@
+    COFF::IMAGE_SCN_CNT_INITIALIZED_DATA | COFF::IMAGE_SCN_MEM_READ,
+    SectionKind::getReadOnly());	  
 }
----------------
I wonder if the stackmap data really needs to be in a loadable section (in other words, memory resident when the code it refers to is executing), or if this should be some kind of client determined thing.

Certainly for LLILC's purposes it doesn't need to be loadable, since we'll translate it to another format.

http://reviews.llvm.org/D10680

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list