Re: svn commit: r276499 - head/sys/conf

2015-01-02 Thread Warner Losh
I’ll back them out in one commit, and commit the fix in the next, so there will be a one commit window. Warner > On Jan 2, 2015, at 9:23 PM, Dimitry Andric wrote: > > Sure, I don't have any problem with backing this out for now. For clang > 3.5.0 users, it should not matter too much that GNU

Re: svn commit: r276499 - head/sys/conf

2015-01-02 Thread Dimitry Andric
Sure, I don't have any problem with backing this out for now. For clang 3.5.0 users, it should not matter too much that GNU as is run for just a few files. -Dimitry > On 03 Jan 2015, at 05:14, Warner Losh wrote: > > Conditioning on the compiler version is actually rather easy, > especially sin

Re: svn commit: r276499 - head/sys/conf

2015-01-02 Thread Warner Losh
Conditioning on the compiler version is actually rather easy, especially since the number of affected files is so small. It’s an issue when people build the old way on old systems, which is still a lot more popular than you might think. MFC has nothing to do with the problem. Over the years we’ve f

Re: svn commit: r276499 - head/sys/conf

2015-01-02 Thread Dimitry Andric
Hm, conditionalizing this on the compiler version is rather ugly. Isn't this only relevant when we do an MFC? But maybe it is indeed better to have the same Makefile for different toolchain envronments. -Dimitry > On 02 Jan 2015, at 18:33, Warner Losh wrote: > > This breaks building the kerne

Re: svn commit: r276499 - head/sys/conf

2015-01-02 Thread Warner Losh via svn-src-head
This breaks building the kernel with clang 3.4.1 Warner > On Jan 1, 2015, at 3:07 AM, Roman Divacky wrote: > > Author: rdivacky > Date: Thu Jan 1 10:07:57 2015 > New Revision: 276499 > URL: https://svnweb.freebsd.org/changeset/base/276499 > > Log: > LLVM integrated assembler can assemble th

svn commit: r276499 - head/sys/conf

2015-01-01 Thread Roman Divacky
Author: rdivacky Date: Thu Jan 1 10:07:57 2015 New Revision: 276499 URL: https://svnweb.freebsd.org/changeset/base/276499 Log: LLVM integrated assembler can assemble these files just fine. Modified: head/sys/conf/Makefile.amd64 head/sys/conf/Makefile.i386 head/sys/conf/Makefile.pc98 Mod