[PATCH] Enable all targets on Windows (MSVC)

Anton Korobeynikov anton at korobeynikov.info
Fri Apr 12 07:16:09 PDT 2013


Hi Tim,

Generally, looks reasonable to me. Do you have any numbers of hand,
though? E.g. differences in build times?

On Fri, Apr 12, 2013 at 6:04 PM, Tim Northover <t.p.northover at gmail.com> wrote:
> 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.
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>



-- 
With best regards, Anton Korobeynikov
Faculty of Mathematics and Mechanics, Saint Petersburg State University



More information about the llvm-commits mailing list