[PATCH] Enable all targets on Windows (MSVC)

Tim Northover t.p.northover at gmail.com
Fri Apr 12 07:04:17 PDT 2013


Hi all,

I'd like to enable all targets by default when building LLVM for MSVC.
The different behaviour seems to date from 2008, and today I don't
think there's any reason to treat it specially.

When Keith asked in
http://comments.gmane.org/gmane.comp.compilers.llvm.devel/60916, the
main reason given for the special case was that Windows support used
to be much more broken. These days the tests seem to pass for all
backends, and certainly should!

Build time will obviously be affected, but the same consideration
applies to all hosts and hasn't caused us to limit the targets
elsewhere.

OK to commit?

Tim.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: enable-all-targets.diff
Type: application/octet-stream
Size: 1495 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130412/6c35f798/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: enable-all-targets-clang.diff
Type: application/octet-stream
Size: 1190 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130412/6c35f798/attachment-0001.obj>


More information about the cfe-commits mailing list