[Beignet] [PATCH] replace hash_map with map

Guo, Yejun yejun.guo at intel.com
Thu Dec 25 21:26:19 PST 2014


Hash_map is defined inside beignet with std::hash, std::equal_to and std::tr1::unordered_map.  Std::hash is not supported by gcc 4.4.7, for example, which is the default compiler of centos 6.5.

-----Original Message-----
From: Beignet [mailto:beignet-bounces at lists.freedesktop.org] On Behalf Of Andi Kleen
Sent: Friday, December 26, 2014 1:17 PM
To: beignet at lists.freedesktop.org
Subject: Re: [Beignet] [PATCH] replace hash_map with map

Guo Yejun <yejun.guo at intel.com> writes:

> there is no strong evidence to show hash_map makes better performance 
> for beignet, since hash_map requires std::hash which is not supported 
> in some g++ old versions, so replace hash_map with map.

Are you sure? What g++ version is that?

AFAIK hash_map has been always in libstdc++ since it exist, as it always was in the SGI STL.

-Andi

--
ak at linux.intel.com -- Speaking for myself only

_______________________________________________
Beignet mailing list
Beignet at lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/beignet


More information about the Beignet mailing list