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.3rd patch http://github.com/Bigcheese/llvm-mirror/commit/061848bca977d5d84b296aa91afd9a3a3dac0093 And patch attached. I'll wait on posting the 4th and 5th (making the X86 back end actually use this stuff on windows and tests) patches until 1 & 2 are committed and this one is reviewed. - Michael Spencer -------------- next part -------------- A non-text attachment was scrubbed... Name: ms-coff-patch-3.patch Type: application/octet-stream Size: 10707 bytes Desc: not available URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20100703/c00edb05/attachment.obj>
This probably needs to be slightly tweaked to work with mainline. I don't see anything objectionable, but I think Daniel needs to review this one. Thanks for working on this Michael! -Chris On Jul 3, 2010, at 3:19 AM, 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. > > 3rd patch > > http://github.com/Bigcheese/llvm-mirror/commit/061848bca977d5d84b296aa91afd9a3a3dac0093 > > And patch attached. > > I'll wait on posting the 4th and 5th (making the X86 back end actually > use this stuff on windows and tests) patches until 1 & 2 are > committed and this one is reviewed. > > - Michael Spencer > <ms-coff-patch-3.patch>_______________________________________________ > LLVM Developers mailing list > LLVMdev at cs.uiuc.edu http://llvm.cs.uiuc.edu > http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev
On Sun, Jul 11, 2010 at 6:10 PM, Chris Lattner <clattner at apple.com> wrote:> This probably needs to be slightly tweaked to work with mainline. I don't see anything objectionable, but I think Daniel needs to review this one.Updated patch to work with mainline. http://github.com/Bigcheese/llvm-mirror/commit/d19a4c82c18afc4830c09b70f02d162292231c94 - Michael Spencer -------------- next part -------------- A non-text attachment was scrubbed... Name: ms-coff-patch-3.1.patch Type: application/octet-stream Size: 13980 bytes Desc: not available URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20100714/c7586ad2/attachment.obj>