Req #69703 [Opn->Asn]: 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: Open +Status: Assigned Type: Feature/Change Request Package: Performance problem Operating System: Linux PHP Version: 5.6Git-2015-05-25 (Git) -Assigned To: +Assigned To: dmitry Block user comment: N Private report: N New Comment: Dmitry, can you verify if this patch can safely go into 5.6? It is obsolete for master it seems Previous Comments: ------------------------------------------------------------------------ [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