Req #69703 [Asn]: Use __builtin_clzl on PowerPC

Edit report at https://bugs.php.net/bug.php?id=69703&edit=1 ID: 69703 Updated by: dmitry@php.net Reported by: dja at axtens dot net Summary: Use __builtin_clzl on PowerPC Status: Assigned Type: Feature/Change Request Package: Performance problem Operating System: Linux PHP Version: 5.6Git-2015-05-25 (Git) Assigned To: dmitry Block user comment: N Private report: N New Comment: Looks fine. Previous Comments: ------------------------------------------------------------------------ [2015-05-28 08:25:09] kalle@php.net Dmitry, can you verify if this patch can safely go into 5.6? It is obsolete for master it seems ------------------------------------------------------------------------ [2015-05-25 01:53:04] dja at axtens dot net Description: ------------ Zend/zend_alloc.c uses __builtin_clzl in zend_mm_high_bit and zend_mm_low_but, but only if (defined(__arm__) || defined(__aarch64__)). This is overly restrictive: it's also usable if __defined__(__powerpc__). This causes a bump to speed on POWER systems: previously zend_mm_high_bit shows up on perf reports, with this patch the functions drop out of the profile entirely. ------------------------------------------------------------------------ -- Edit this bug report at https://bugs.php.net/bug.php?id=69703&edit=1