[PATCH] Enable building LTO on WIN32
Greg_Bedwell at sn.scee.net
Greg_Bedwell at sn.scee.net
Mon Sep 30 07:57:40 PDT 2013
> We should switch to in source annotations at some point, but we should
> do so for windows and ELF/MachO at the same time. Using the .def for
> now is OK.
>
Thanks. I agree with this entirely.
> Patch LGTM.
Please can you commit for me? I'm still working towards commit access. Do
you want me to rebase against top of trunk?
-Greg
Rafael EspĂndola <rafael.espindola at gmail.com> wrote on 30/09/2013
14:46:37:
> From: Rafael EspĂndola <rafael.espindola at gmail.com>
> To: Greg_Bedwell at sn.scee.net
> Cc: llvm-commits <llvm-commits at cs.uiuc.edu>
> Date: 30/09/2013 14:46
> Subject: Re: [PATCH] Enable building LTO on WIN32
>
> On 30 September 2013 09:31, <Greg_Bedwell at sn.scee.net> wrote:
> > Hi,
> >
> >> Instead of creating a .def file, can't we annotate the source with
> > dllexport?
> >>
> >
> > I'm happy to make that change if that's the way that is generally
> > preferred. In general for Windows-only code I'd choose dllexport
> > annotations over creating a .def file to keep the information
> > contained/documented in the header. My concern here was that the
Makefile
> > builds already use the lto.exports file as the canonical list of
exported
> > functions so by using that as the basis for the Windows DLL exports
we've
> > only got the information in a single place so there's no ongoing work
to
> > keep them in sync (beyond having to rerun CMake - a further
improvement
> > might be to see if we can move that process into build time rather
than
> > CMake time).
> >
> > I'll go with whatever option is preferred. Personally, for the above
> > reason I prefer the def file approach in this case but I don't have a
> > particularly strong opinion if the exported interface is expected to
> > generally remain fairly stable.
>
> OK, I agree.
>
> We should switch to in source annotations at some point, but we should
> do so for windows and ELF/MachO at the same time. Using the .def for
> now is OK.
>
> Patch LGTM.
>
> Cheers,
> Rafael
**********************************************************************
This email and any files transmitted with it are confidential and intended
solely for the use of the individual or entity to whom they are addressed.
If you have received this email in error please notify postmaster at scee.net
This footnote also confirms that this email message has been checked for
all known viruses.
Sony Computer Entertainment Europe Limited
Registered Office: 10 Great Marlborough Street, London W1F 7LP, United
Kingdom
Registered in England: 3277793
**********************************************************************
P Please consider the environment before printing this e-mail
More information about the llvm-commits
mailing list