Req #69703 [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: Closed Type: Feature/Change Request Package: Performance problem Operating System: Linux PHP Version: 5.6Git-2015-05-25 (Git) Assigned To: kalle Block user comment: N Private report: N New Comment: I committed this only to 5.6, as 5.5 is going to security only mode in 22 days (unless Julien decides it should go through). This should be available in 5.6.10 (on 11th of June) Previous Comments: ------------------------------------------------------------------------ [2015-05-28 12:31:29] kalle@php.net 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 ------------------------------------------------------------------------ [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