[LLVMdev] Win32 COFF Support - Patch 2
clattner at apple.com
Sun Jul 11 15:09:48 PDT 2010
Looks good, committed in r108103. I changes various _OS arguments to be named OS. Please don't use underbars as the first character of an identifier, they are a reserved namespace for the compiler implementation (with capital letters).
On Jul 2, 2010, at 9:26 PM, Michael Spencer wrote:
> On Wed, Jun 16, 2010 at 1:46 PM, Daniel Dunbar <daniel at zuster.org> wrote:
>> - Please split the patch into at least four parts:
>> (1) Add WinCOFFStreamer, with a stub implementation.
>> (2) Add WinCOFFObjectWriter, with a stub implementation.
>> (3) Fill in WinCOFFStreamer.
>> (4) Fill in WinCOFFObjectWriter.
> Here's the 2nd patch.
> And a git patch is attached this time (Can't really make an svn style
> patch until the 1st patch is committed).
> - Michael Spencer
> LLVM Developers mailing list
> LLVMdev at cs.uiuc.edu http://llvm.cs.uiuc.edu
More information about the llvm-dev