Req #69703 [Asn->Csd]: Use __builtin_clzl on PowerPC

Edit report at https://bugs.php.net/bug.php?id=69703&edit=1 ID: 69703 Updated by: kalle@php.net Reported by: dja at axtens dot net Summary: Use __builtin_clzl on PowerPC -Status: Assigned +Status: Closed 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: Automatic comment on behalf of kalle Revision: http://git.php.net/?p=php-src.git;a=commit;h=de696d851f18efb40298d92b83d761a39964b910 Log: Fixed bug #69703 (Use __builtin_clzl on PowerPC) -- Patch by dja at axtens dot net Previous Comments: ------------------------------------------------------------------------ [2015-05-28 08:30:58] dmitry@php.net Looks fine. ------------------------------------------------------------------------ [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