Re: archaic/useless CFLAGS options for x86 boot blocks

2011-09-12 Thread Andriy Gapon
on 12/09/2011 15:26 John Baldwin said the following: > On Monday, September 12, 2011 8:02:30 am Andriy Gapon wrote: >>> I think the patch is >>> fine, and I'd even prefer to go ahead and drop the extra cruft (like >>> removing >>> nops and aligns as well as -mrtd and -mregparm) from the UFS boot2

Re: archaic/useless CFLAGS options for x86 boot blocks

2011-09-12 Thread John Baldwin
On Monday, September 12, 2011 8:02:30 am Andriy Gapon wrote: > > I think the patch is > > fine, and I'd even prefer to go ahead and drop the extra cruft (like > > removing > > nops and aligns as well as -mrtd and -mregparm) from the UFS boot2 as well. > > I personally agree, thank you for this su

Re: archaic/useless CFLAGS options for x86 boot blocks

2011-09-12 Thread Andriy Gapon
on 12/09/2011 14:43 John Baldwin said the following: > I suspect some of the recent changes to shave space down for Clang have made > some of the optimization options no longer necessary. Just a note of all the options in question were added long before clang. > I think the patch is > fine, and

Re: archaic/useless CFLAGS options for x86 boot blocks

2011-09-12 Thread John Baldwin
On Monday, September 12, 2011 3:36:54 am Andriy Gapon wrote: > > This email is in part inspired by the following problem: > http://article.gmane.org/gmane.os.freebsd.current/135292 > So "harmful" could also be added to the subject line. > > So here is my proposal. > > Part I. ZFS and GPT bootbl

Re: archaic/useless CFLAGS options for x86 boot blocks

2011-09-12 Thread Andriy Gapon
on 12/09/2011 10:36 Andriy Gapon said the following: > > This email is in part inspired by the following problem: > http://article.gmane.org/gmane.os.freebsd.current/135292 > So "harmful" could also be added to the subject line. > > So here is my proposal. I would like to clarify that my intenti