Ian,

Yes, I realized there was a problem compling with gcc4 with this code. I fixed
that since then.

Thanks.

On Mon, May 15, 2006 at 10:38:15AM +1000, Ian Wienand wrote:
> On Fri, May 12, 2006 at 08:15:37AM -0700, Stephane Eranian wrote:
> > The new version of the library
> 
> Hi,
> 
> I'm trying to build libpfm-3.2-060512 for my Pentium M laptop and see
> 
> pfmlib_gen_ia32.c: In function 'pfm_gen_ia32_detect':
> pfmlib_gen_ia32.c:117: error: PIC register 'ebx' clobbered in 'asm'
> pfmlib_gen_ia32.c:122: error: PIC register 'ebx' clobbered in 'asm'
> pfmlib_gen_ia32.c:117: error: PIC register 'ebx' clobbered in 'asm'
> pfmlib_gen_ia32.c:122: error: PIC register 'ebx' clobbered in 'asm'
> 
> I think it's just a typo, since you're saving and restoring ebx it
> doesn't need to be listed as clobbered?
> 
> -i
> 
> --- libpfm-3.2-060512/lib/pfmlib_gen_ia32.c   2006-05-13 00:22:45.000000000 
> +1000
> +++ libpfm-3.2-060512-new/lib/pfmlib_gen_ia32.c       2006-05-15 
> 10:34:37.000000000 +1000
> @@ -117,12 +117,12 @@
>       __asm__("pushl %%ebx;cpuid; popl %%ebx"
>                       :"=a" (*eax)
>                       : "a" (op)
> -                     : "ecx", "edx", "ebx");
> +                     : "ecx", "edx");
>  
>       __asm__("pushl %%ebx;cpuid; movl %%ebx, %%eax;popl %%ebx"
>                       :"=a" (*ebx)
>                       : "a" (op)
> -                     : "ecx", "edx", "ebx");
> +                     : "ecx", "edx");
>  }
>  
>  /*

-- 

-Stephane
_______________________________________________
perfmon mailing list
[email protected]
http://www.hpl.hp.com/hosted/linux/mail-archives/perfmon/

Reply via email to