From 729b58296282da00fb9c0f92db2e2e8a8347d431 Mon Sep 17 00:00:00 2001 From: Mike Klein Date: Mon, 28 Nov 2016 18:23:23 -0500 Subject: Consistent naming. For stages that have {r,g,b,a} and {dr,dg,db,da} versions, name the {r,g,b,a} one "foo" and the {dr,dg,db,da} on "foo_d". The {r,g,b,a} registers are the ones most commonly used and fastest, so they get short ordinary names, and the d-registers are less commonly used and sometimes slower, so they get a suffix. Some stages naturally opearate on all 8 registers (the xfermodes, accumulate). These names for those look fine and aren't ambiguous. Also, a bit more re-arrangement in _opts.h. CQ_INCLUDE_TRYBOTS=skia.primary:Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD Change-Id: Ia20029247642798a60a2566e8a26b84ed101dbd0 Reviewed-on: https://skia-review.googlesource.com/5291 Reviewed-by: Mike Klein Commit-Queue: Mike Klein --- bench/SkRasterPipelineBench.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'bench/SkRasterPipelineBench.cpp') diff --git a/bench/SkRasterPipelineBench.cpp b/bench/SkRasterPipelineBench.cpp index 84c0930466..64e59c8b54 100644 --- a/bench/SkRasterPipelineBench.cpp +++ b/bench/SkRasterPipelineBench.cpp @@ -37,13 +37,13 @@ public: void* dst_ctx = dst; SkRasterPipeline p; - p.append(SkRasterPipeline::load_s_8888, &src_ctx); - p.append(SkRasterPipeline::from_srgb_s); + p.append(SkRasterPipeline::load_8888, &src_ctx); + p.append(SkRasterPipeline::from_srgb); p.append(SkRasterPipeline::scale_u8, &mask_ctx); if (kF16) { - p.append(SkRasterPipeline::load_d_f16, &dst_ctx); + p.append(SkRasterPipeline::load_f16_d, &dst_ctx); } else { - p.append(SkRasterPipeline::load_d_8888, &dst_ctx); + p.append(SkRasterPipeline::load_8888_d, &dst_ctx); p.append(SkRasterPipeline::from_srgb_d); } p.append(SkRasterPipeline::srcover); -- cgit v1.2.3