GitHub PushEvent at 2014-09-18 23:41:02
Repository: android_external_skia
Actor: temasek
Payload: {"shas":[["268beb454aa3bc63a59148491f9a075a00e3e82a","5c2bf027dc5cc7ef186314391646e49f6866dc00@chromium.org","ARM Skia NEON patches - 20 - New improved BitmapProcState code\n\nBitmapProcState: new factorised code\n\nThis one basically factorises the clamp and repeat transformations with\nsome performance improvements. It has the benefit of being faster, much\neasier to maintain (nearly three times less code for more work\ndone :-)), and more complete (all persp transformations weren't optimised\nin the previous version).\n\nIt also introduces the use of can_truncate_to_fixed_for_decal where\nuseful.\n\nThe effect on benchmarks ranges from a 5% penalty to a 25% gain on a\nCortex-A9 and from a 5% penalty to a 100% gain on a Cortex-A15.\n\nSigned-off-by: Kévin PETIT <kevin.petit@arm.com>\n\nBUG=\nR=djsollen@google.com, mtklein@google.com, luisjoseromeroesclusa@hotmail.com, reed@google.com\n\nAuthor: kevin.petit.arm@gmail.com\n\nReview URL: https://codereview.chromium.org/23835006\n\nChange-Id: Ifb1a610730548f6e487dbb60bebcf69150186ed1\ngit-svn-id: http://skia.googlecode.com/svn/trunk@13218 2bbb7eff-a529-9590-31e7-b0007b416f81\n\nPatchset: 1\nhttp://review.cyanogenmod.org/#/c/73453/","commit-bot@chromium.org",true],["d48151300a076257c4c3763d7b923c8f3d66501a","5c2bf027dc5cc7ef186314391646e49f6866dc00@chromium.org","ARM Skia NEON patches - 27 - S32A_D565_Blend\n\nBlitRow565: new intrinsics version of S32A_D565_Blend\n\nThis new version is basically a rewrite of the existing code with\na few speed and accuracy improvements. There is a switch to enable\npixel perfect results at the cost of a (quite big) decrease of\nperformances (disabled in this patch).\n\nHere are the benchmark results (speedup vs. existing code):\n\n+-------+------------+------------+\n| count | Cortex -A9 | Cortex-A15 |\n+-------+------------+------------+\n| 1     | +103.6%    | +12%       |\n+-------+------------+------------+\n| 2     | +3.6%      | +21.6%     |\n+-------+------------+------------+\n| 4     | +0.8%      | -0.8%      |\n+-------+------------+------------+\n| 8     | +3.9%      | -1%        |\n+-------+------------+------------+\n| 16    | +14.7%     | +5.7%      |\n+-------+------------+------------+\n| 64    | +18.1%     | +13.2%     |\n+-------+------------+------------+\n| 256   | +16.3%     | +27.4%     |\n+-------+------------+------------+\n| 1024  | +78.2%     | +17.4%     |\n+-------+------------+------------+\n\nSigned-off-by: Kévin PETIT <kevin.petit@arm.com>\n\nBUG=skia:\nR=djsollen@google.com, mtklein@google.com, halcanary@google.com\n\nAuthor: kevin.petit@arm.com\n\nReview URL: https://codereview.chromium.org/156113005\n\nChange-Id: I92a1a6dfdb813b77ba05b245a3fdbb5e02458981\ngit-svn-id: http://skia.googlecode.com/svn/trunk@13438 2bbb7eff-a529-9590-31e7-b0007b416f81\n\nPatchset: 1\nhttp://review.cyanogenmod.org/#/c/73454/","commit-bot@chromium.org",true],["879a8b195a8c2ec27c3fe999435fdedd28136b4c","5c2bf027dc5cc7ef186314391646e49f6866dc00@chromium.org","ARM Skia NEON patches - 20bis - BitmapProcState accurracy\n\nUse SkFractionalInt for some calculations to improve accuracy.\n\nSigned-off-by: Kévin PETIT <kevin.petit@arm.com>\n\nBUG=skia:2175\nNOTRY=true\nR=djsollen@google.com, mtklein@google.com, hshi@chromium.org\n\nAuthor: kevin.petit@arm.com\n\nReview URL: https://codereview.chromium.org/167433002\n\nChange-Id: I2452ff401cc584084f44ec3a27240aa564eb19bc\ngit-svn-id: http://skia.googlecode.com/svn/trunk@13518 2bbb7eff-a529-9590-31e7-b0007b416f81\n\nPatchset: 1\nhttp://review.cyanogenmod.org/#/c/73455/","commit-bot@chromium.org",true],["bb53092e3778c457e3364b85738fb3f33ab11842","5c2bf027dc5cc7ef186314391646e49f6866dc00@chromium.org","ARM Skia NEON patches - 12 - S32_Blend\n\nBlitrow32: S32_Blend fix and little speed improvement\n\n- the results are now exactly similar as the C code\n- the speed has improved, especially for small values of count\n\n+-------+-----------+------------+\n| count | Cortex-A9 | Cortex-A15 |\n+-------+-----------+------------+\n| 1     | +30%      | +18%       |\n+-------+-----------+------------+\n| 2     | 0         | 0          |\n+-------+-----------+------------+\n| 4     | - <1%     | +14%       |\n+-------+-----------+------------+\n| > 4   | -0.5..+5% | -0.5..+4%  |\n+-------+-----------+------------+\n\nSigned-off-by: Kévin PETIT <kevin.petit@arm.com>\n\nBUG=skia:\n\nCommitted: http://code.google.com/p/skia/source/detail?r=13532\n\nR=djsollen@google.com, mtklein@google.com\n\nAuthor: kevin.petit@arm.com\n\nReview URL: https://codereview.chromium.org/158973002\n\nChange-Id: I75a4d2a31313664b4df057ea58821899646a1577\ngit-svn-id: http://skia.googlecode.com/svn/trunk@13543 2bbb7eff-a529-9590-31e7-b0007b416f81\n\nPatchset: 1\nhttp://review.cyanogenmod.org/#/c/73456/","commit-bot@chromium.org",true],["676e1dbc4a22a339b9c83d89416564cf6d327b51","5c2bf027dc5cc7ef186314391646e49f6866dc00@chromium.org","ARM Skia NEON patches - 25 - S32A_D565_Opaque_Dither clean/bugfix/speed\n\nBlitRow565: S32A_D565_Opaque_Dither: some improvements\n\n    - Supports ARGB and ABGR\n    - Less magic numbers\n    - Reduced instruction count : 5-25% speedup\n    - Fixed indentation, removed some commented and useless code\n\nSigned-off-by: Kévin PETIT <kevin.petit@arm.com>\n\nBUG=skia:\nR=djsollen@google.com, mtklein@google.com\n\nAuthor: kevin.petit@arm.com\n\nReview URL: https://codereview.chromium.org/177963003\n\nChange-Id: I4f6083372e653fda5224b0f74dc5affbaa4c8f8f\ngit-svn-id: http://skia.googlecode.com/svn/trunk@13577 2bbb7eff-a529-9590-31e7-b0007b416f81\n\nPatchset: 1\nhttp://review.cyanogenmod.org/#/c/73457/","commit-bot@chromium.org",true],["d945a80b3f274559248c81605421ee92d61f8242","5c2bf027dc5cc7ef186314391646e49f6866dc00@chromium.org","ARM Skia NEON patches - 22 - S32_D565_Blend\n\nBlitRow565: new NEON version of S32_D565_Blend\n\nThis new implementation brings a good speedup in most cases and\ngives exact results (removes one mismatch in gm).\n\nHere are the benchmark results (speedup vs. existing S32A_D565_Blend):\n\n+-------+-----------+------------+\n| count | Cortex-A9 | Cortex-A15 |\n+-------+-----------+------------+\n| 1     | -26,7%    | -27,5%     |\n+-------+-----------+------------+\n| 2     | 0%        | +53%       |\n+-------+-----------+------------+\n| 4     | +38,3%    | +26,5%     |\n+-------+-----------+------------+\n| 8     | +10,9%    |  -4,5%     |\n+-------+-----------+------------+\n| 16    | +18,2%    | +1,6%      |\n+-------+-----------+------------+\n| 64    | +22,3%    | +8,75%     |\n+-------+-----------+------------+\n| 256   | +12,3%    | +11,2%     |\n+-------+-----------+------------+\n| 1024  | +79,2%    | +10,9%     |\n+-------+-----------+------------+\n\nSigned-off-by: Kévin PETIT <kevin.petit@arm.com>\n\nBUG=skia:\nR=djsollen@google.com, mtklein@google.com\n\nAuthor: kevin.petit@arm.com\n\nReview URL: https://codereview.chromium.org/181523002\n\nChange-Id: I9307ea4ee185e0319aed20ef9708cef5552c561b\ngit-svn-id: http://skia.googlecode.com/svn/trunk@14103 2bbb7eff-a529-9590-31e7-b0007b416f81\n\nPatchset: 1\nhttp://review.cyanogenmod.org/#/c/73458/","commit-bot@chromium.org",true],["83864a61c4b009ee1f07abacb5a9e32a80bd4248","5c2bf027dc5cc7ef186314391646e49f6866dc00@chromium.org","ARM Skia NEON patches - 36 - Color32\n\nConvert Color32 to intrinsics\n\nThis change is performance-neutral for high values of count and is\na big improvement for values smaller than 64.\n\nSigned-off-by: Kévin PETIT <kevin.petit@arm.com>\n\nBUG=skia:\nR=djsollen@google.com, mtklein@google.com, borenet@google.com\n\nAuthor: kevin.petit@arm.com\n\nReview URL: https://codereview.chromium.org/258173005\n\nChange-Id: Id7173a61efc6ebf8812149d2084e2b27c07fec52\ngit-svn-id: http://skia.googlecode.com/svn/trunk@14435 2bbb7eff-a529-9590-31e7-b0007b416f81\n\nPatchset: 1\nhttp://review.cyanogenmod.org/#/c/73459/","commit-bot@chromium.org",true],["528f38545384bb9668a952c260e1bd2dab36934b","5c2bf027dc5cc7ef186314391646e49f6866dc00@chromium.org","Remove the unused SkCachePreload_arm\n\nSigned-off-by: Kévin PETIT <kevin.petit@arm.com>\n\nBUG=skia:\nR=djsollen@google.com, mtklein@google.com\n\nAuthor: kevin.petit@arm.com\n\nReview URL: https://codereview.chromium.org/263553008\n\nChange-Id: I7312ca230cd98ec335970c4f503b3b04d2ec7db3\ngit-svn-id: http://skia.googlecode.com/svn/trunk@14456 2bbb7eff-a529-9590-31e7-b0007b416f81\n\nPatchset: 1\nhttp://review.cyanogenmod.org/#/c/73460/","commit-bot@chromium.org",true],["6852456379f0e93fa4e67c24feff133756843633","5c2bf027dc5cc7ef186314391646e49f6866dc00@chromium.org","Always inline Filter_32_*_neon functions\n\nThe functions are rather performance critical and already marked\n'inline'. However, Chrome for Android will not have these functions\ninlined due to it being compiled with -Os and a small -finline-limit.\n\nThis avoids one call in the filtering functions.\n\nDoes not increase the library size.\n\nBUG=chromium:363073\nR=mtklein@google.com\n\nAuthor: kkinnunen@nvidia.com\n\nReview URL: https://codereview.chromium.org/280403005\n\nChange-Id: I3a1191a3b6f0afe8c035ab0af745280883680efc\ngit-svn-id: http://skia.googlecode.com/svn/trunk@14709 2bbb7eff-a529-9590-31e7-b0007b416f81\n\nPatchset: 1\nhttp://review.cyanogenmod.org/#/c/73461/","commit-bot@chromium.org",true],["43dfa64e4a1e6ffd0eb7a982fbee65cb315e49fb","9fd8de5fc2a7c2c0d469b2fff1afde4e5def37ba@mozilla.com","Fix SkBlitRow_opts_arm so that it works on ARM v4t.\nOriginal Mozilla bug: https://bugzilla.mozilla.org/show_bug.cgi?id=901208\n\nR=reed@google.com, mtklein@google.com, reed1\nBUG=skia:\n\nAuthor: george@mozilla.com\n\nReview URL: https://codereview.chromium.org/337853003\n\nChange-Id: I822ef24123f2ae10815d99f25d07ee06e2c47af9\n\nPatchset: 1\nhttp://review.cyanogenmod.org/#/c/73462/","george",true],["45bc70981fd3e5fe0c842dff123666a318988653","6d299cc47d6d35c32bbc7a3351feef497705fdf7@stericsson.com","Add optimization of Skia S32A_Opaque blitter\n\nAdds optimization of Skia S32A_Opaque_BlitRow32 blitter using ARM NEON\ninstruction set. Special cases for when alpha is zero or opaque.\nImproves performance platform wide.\n\nChange-Id: I0ffeb23b128e61cfe581ad121f227631d2918686\nSigned-off-by: Henrik Smiding <henrik.smiding@stericsson.com>\nSigned-off-by: Patrik Ryd <patrik.ryd@stericsson.com>\n\nPatchset: 1\nhttp://review.cyanogenmod.org/#/c/73463/","Henrik Smiding",true],["c093326e03b8a4a1c76ea137a24ed49137c6abf2","6d299cc47d6d35c32bbc7a3351feef497705fdf7@stericsson.com","Add optimization of Skia S32A_Blend blitter\n\nAdds optimization of Skia S32A_Blend_BlitRow32 blitter using ARM NEON\ninstruction set. Special cases for when pixel alpha is zero.\nImproves performance platform wide.\n\nChange-Id: I6c8bf8a9525838682206ebd139855354d6b3a563\nSigned-off-by: Henrik Smiding <henrik.smiding@stericsson.com>\nSigned-off-by: Patrik Ryd <patrik.ryd@stericsson.com>\n\nPatchset: 1\nhttp://review.cyanogenmod.org/#/c/73464/","Henrik Smiding",true],["fd11d4c89a92722ff4164cc7db31b1c49cdc33f6","6d299cc47d6d35c32bbc7a3351feef497705fdf7@stericsson.com","Add optimized S32A_D565_Opaque_Dither blitter.\n\nAdds optimization of Skia S32A_D565_Opaque_Dither blitter using\nARM NEON instruction set.\nImproves performance in browser.\nChange-Id: Ie20fb3cf3ad9a60f01b79e3ed2ddba7651bcf013\nSigned-off-by: Henrik Smiding <henrik.smiding@stericsson.com>\nSigned-off-by: Patrik Ryd <patrik.ryd@stericsson.com>\n\nPatchset: 1\nhttp://review.cyanogenmod.org/#/c/73465/","Henrik Smiding",true],["300a2539509b4ce181411d820e72937d81a3040b","6d299cc47d6d35c32bbc7a3351feef497705fdf7@intel.com","Modify sample buffer size for larger displays\n\nBZ: 146848\n\nIncreases the intermediate buffer size for sample pixel indexes,\nused in the sample proc function calls. If the operation is bigger\nthan the buffer it's split into multiple calls, creating overhead.\nThis would especially impact the performance of SIMD optimizations.\n\nChange-Id: I44d0723cb26372050741e28fc14f676ddcd56d77\nCategory: aosp improvement\nDomain: AOSP-Others\nOrigin: internal\nUpstream-Candidate: yes\nSigned-off-by: Henrik Smiding <henrik.smiding@intel.com>\nReviewed-on: https://android.intel.com/163478\nTested-by: sys_abtbuild <sys_abtbuild@intel.com>\nReviewed-by: Landberg, Joakim <joakim.landberg@intel.com>\nReviewed-by: Chen, HuaqiangX <huaqiangx.chen@intel.com>\nReviewed-by: Fagerstedt, Axel <axel.fagerstedt@intel.com>\n\nPatchset: 1\nhttp://review.cyanogenmod.org/#/c/73466/","Henrik Smiding",true]],"size":14,"ref":"refs/heads/cm-11.0","head":"300a2539509b4ce181411d820e72937d81a3040b"}