[PATCH] Preliminary support for dynamically loadable coff objects
David Majnemer
david.majnemer at gmail.com
Tue Feb 24 11:05:43 PST 2015
REPOSITORY
rL LLVM
================
Comment at: lib/Object/COFFObjectFile.cpp:376
@@ -366,3 +375,3 @@
const coff_section *Sec = toSec(Ref);
- return Sec->Characteristics & COFF::IMAGE_SCN_CNT_UNINITIALIZED_DATA;
+ return Sec->SizeOfRawData == 0;
}
----------------
AndyAyers wrote:
> majnemer wrote:
> > AndyAyers wrote:
> > > majnemer wrote:
> > > > What's the rationale for this change?
> > > I thought it better matched the intent of the comment in ObjectFile.h:
> > >
> > > // A section is 'virtual' if its contents aren't present in the object image.
> > >
> > > For coff this means the section has no raw data, regardless of section characteristics. In practice it probably doesn't matter so I'd be happy to revert this if you prefer.
> > Compiling `char x[300];` as C++ with MSVC 2015 results in a .bss section with `SizeOfRawData` set to 0x12c.
> >
> > The relevant text from the COFF spec is:
> > > The size of the section (for object files) or the size of the initialized data on disk (for image files).
> >
> > Heh, I guess my formulation of `isSectionVirtual` was more appropriate for object files while yours is better for executables.
> Perhaps a better check is if PointerToRawData is zero, this should work in both obj and pe cases?
Yes, I think that would work perfectly.
http://reviews.llvm.org/D7793
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list