On Wed, Mar 11, 2009 at 1:41 PM, David Cournapeau <courn...@gmail.com>wrote:

> On Thu, Mar 12, 2009 at 3:15 AM, Ryan May <rma...@gmail.com> wrote:
> > Hi,
> >
> > I noticed the following in numpy/distutils/system_info.py while trying to
> > get numpy to build against MKL:
> >
> >             if cpu.is_Itanium():
> >                 plt = '64'
> >                 #l = 'mkl_ipf'
> >             elif cpu.is_Xeon():
> >                 plt = 'em64t'
> >                 #l = 'mkl_em64t'
> >             else:
> >                 plt = '32'
> >                 #l = 'mkl_ia32'
> >
> > So in the autodetection for MKL, the only way to get plt (platform) set
> to
> > 'em64t' is to test true for a Xeon.  This function returns false on my
> Core2
> > Duo system, even though the platform is very much 'em64t'.  I think that
> > check should instead read:
> >
> > elif cpu.is_Xeon() or cpu.is_Core2():
> >
> > Thoughts?
>
> I think this whole code is inherently fragile. A much better solution
> is to make the build process customization easier and more
> straightforward. Auto-detection will never work well.
>
> David
> _______________________________________________
> Numpy-discussion mailing list
> Numpy-discussion@scipy.org
> http://mail.scipy.org/mailman/listinfo/numpy-discussion
>

Fair enough.

Ryan

-- 
Ryan May
Graduate Research Assistant
School of Meteorology
University of Oklahoma
_______________________________________________
Numpy-discussion mailing list
Numpy-discussion@scipy.org
http://mail.scipy.org/mailman/listinfo/numpy-discussion

Reply via email to