Date: Sat, 14 Oct 1995 08:26:20 +1000 From: Bruce Evans <bde@zeta.org.au> To: current@freebsd.org, imb@scgt.oz.au Subject: Re: microtime.s .. I586 .. Message-ID: <199510132226.IAA22215@godzilla.zeta.org.au>
next in thread | raw e-mail | index | archive | help
>Could someone please add .. >*** microtime.s~ Fri Oct 13 07:11:11 1995 >--- microtime.s Sat Oct 14 03:30:52 1995 >*************** >*** 171,176 **** >--- 171,177 ---- > > ret > >+ #ifdef I586_CPU > .extern _i586_ctr_bias > > ALIGN_TEXT >*************** >*** 181,183 **** >--- 182,185 ---- > sbbl _i586_ctr_bias+4, %edx > divl %ecx # get value in usec > jmp common_microtime >+ #endif >.. for the benefit of those who don't compile for such hardware, I'm surprised anyone compiles without option I586_CPU. The benefits of not using it are tiny, and GENERIC uses it. The above code used to compile, but was only executed on non-penti586's. Now, after yesterday's changes it fails to compile because `i586_ctr_bias' is carefully ifdefed elsewhere and doesn't exist unless I586_CPU is defined. This is perhaps fortunate. Yesterday's changes also broke the clearing of %ecx for the non-i586 case. I hate options. Bruce
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?199510132226.IAA22215>