Re: A suspicious warning in sys/boot/zfs/zfsimpl.c

2012-12-02 Thread Dimitry Andric
On 2012-12-02 01:37, Garrett Cooper wrote: On Sat, Dec 1, 2012 at 3:05 AM, Andriy Gapon wrote: I believe that there is no actual problem there. It's probably bugs with clang dealing with alignment problems. Which bugs? These warnings (and others) go largely unnoticed because of the

Re: A suspicious warning in sys/boot/zfs/zfsimpl.c

2012-12-02 Thread Dimitry Andric
On 2012-12-01 12:05, Andriy Gapon wrote: on 01/12/2012 12:59 Garrett Cooper said the following: On Mon, Jul 2, 2012 at 6:21 PM, Taku YAMAMOTO wrote: When I built the world as of r237813, clang reported a warning which caught my attention. ===> sys/boot/zfs (all) clang -O2 -pipe -march=pentiu

Re: A suspicious warning in sys/boot/zfs/zfsimpl.c

2012-12-01 Thread Garrett Cooper
On Sat, Dec 1, 2012 at 3:05 AM, Andriy Gapon wrote: > I believe that there is no actual problem there. It's probably bugs with clang dealing with alignment problems. These warnings (and others) go largely unnoticed because of the fact that -Werror isn't on on sys/boot. I filed kern/173932 fo

Re: A suspicious warning in sys/boot/zfs/zfsimpl.c

2012-12-01 Thread Garrett Cooper
On Mon, Jul 2, 2012 at 6:21 PM, Taku YAMAMOTO wrote: > When I built the world as of r237813, clang reported a warning which > caught my attention. > > ===> sys/boot/zfs (all) > clang -O2 -pipe -march=pentium4 -DBOOTPROG=\"zfsloader\" > -I/usr/src/sys/boot/zfs/../common -I/usr/src/sys/boot/zfs/..

Re: A suspicious warning in sys/boot/zfs/zfsimpl.c

2012-12-01 Thread Andriy Gapon
on 01/12/2012 12:59 Garrett Cooper said the following: > On Mon, Jul 2, 2012 at 6:21 PM, Taku YAMAMOTO wrote: >> When I built the world as of r237813, clang reported a warning which >> caught my attention. >> >> ===> sys/boot/zfs (all) >> clang -O2 -pipe -march=pentium4 -DBOOTPROG=\"zfsloader\"