[Mesa-dev] [PATCH 0/1] Code formatting changes in swr driver

Cherniak, Bruce bruce.cherniak at intel.com
Tue Jun 12 15:17:37 UTC 2018


Reviewed-by: Bruce Cherniak <bruce.cherniak at intel.com>
(and thanks for not emailing such a huge patch)

> On Jun 8, 2018, at 3:01 PM, Alok Hota <alok.hota at intel.com> wrote:
> 
> Sorry for the large patch - the changes are all whitespace and code
> formatting for the swr driver and its rasterizer. 
> The patch file itself is available on Github as a Gist:
> https://gist.github.com/ahota/34210949630618f684babc4cfa240aa5
> 
> This is to prevent the full 3.1 MB of it being sent out to everyone's
> email.
> 
> Alok Hota (1):
>  swr/rast: Clang-Format most rasterizer source code
> 
> .../swr/rasterizer/archrast/archrast.cpp      |   415 +-
> .../swr/rasterizer/archrast/archrast.h        |    59 +-
> .../swr/rasterizer/archrast/eventmanager.h    |    57 +-
> .../codegen/templates/gen_ar_event.cpp        |    65 +-
> .../codegen/templates/gen_ar_event.hpp        |    70 +-
> .../codegen/templates/gen_ar_eventhandler.hpp |    66 +-
> .../templates/gen_ar_eventhandlerfile.hpp     |    91 +-
> .../codegen/templates/gen_backend.cpp         |     6 +-
> .../codegen/templates/gen_builder.hpp         |     8 +-
> .../codegen/templates/gen_header_init.hpp     |     9 +-
> .../codegen/templates/gen_knobs.cpp           |    82 +-
> .../rasterizer/codegen/templates/gen_knobs.h  |    74 +-
> .../rasterizer/codegen/templates/gen_llvm.hpp |    69 +-
> .../codegen/templates/gen_rasterizer.cpp      |     8 +-
> .../drivers/swr/rasterizer/common/formats.cpp | 13725 +++++++++-------
> .../drivers/swr/rasterizer/common/formats.h   |   403 +-
> .../drivers/swr/rasterizer/common/intrin.h    |   100 +-
> .../drivers/swr/rasterizer/common/isa.hpp     |    94 +-
> .../drivers/swr/rasterizer/common/os.cpp      |   161 +-
> .../drivers/swr/rasterizer/common/os.h        |   153 +-
> .../swr/rasterizer/common/rdtsc_buckets.cpp   |    95 +-
> .../swr/rasterizer/common/rdtsc_buckets.h     |    87 +-
> .../rasterizer/common/rdtsc_buckets_shared.h  |    88 +-
> .../swr/rasterizer/common/simd16intrin.h      |   308 +-
> .../swr/rasterizer/common/simdintrin.h        |   446 +-
> .../drivers/swr/rasterizer/common/simdlib.hpp |   532 +-
> .../swr/rasterizer/common/simdlib_128_avx.inl |   409 +-
> .../rasterizer/common/simdlib_128_avx2.inl    |    64 +-
> .../rasterizer/common/simdlib_128_avx512.inl  |   322 +-
> .../common/simdlib_128_avx512_core.inl        |   219 +-
> .../common/simdlib_128_avx512_knights.inl     |    43 +-
> .../swr/rasterizer/common/simdlib_256_avx.inl |   625 +-
> .../rasterizer/common/simdlib_256_avx2.inl    |   215 +-
> .../rasterizer/common/simdlib_256_avx512.inl  |   318 +-
> .../common/simdlib_256_avx512_core.inl        |   142 +-
> .../common/simdlib_256_avx512_knights.inl     |    43 +-
> .../rasterizer/common/simdlib_512_avx512.inl  |   511 +-
> .../common/simdlib_512_avx512_core.inl        |   235 +-
> .../common/simdlib_512_avx512_knights.inl     |   181 +-
> .../common/simdlib_512_avx512_masks.inl       |    42 +-
> .../common/simdlib_512_avx512_masks_core.inl  |    42 +-
> .../simdlib_512_avx512_masks_knights.inl      |    42 +-
> .../swr/rasterizer/common/simdlib_512_emu.inl |   755 +-
> .../common/simdlib_512_emu_masks.inl          |    43 +-
> .../rasterizer/common/simdlib_interface.hpp   |    42 +-
> .../swr/rasterizer/common/simdlib_types.hpp   |   338 +-
> .../swr/rasterizer/common/swr_assert.cpp      |   192 +-
> .../swr/rasterizer/common/swr_assert.h        |   197 +-
> .../drivers/swr/rasterizer/core/api.cpp       |  1083 +-
> src/gallium/drivers/swr/rasterizer/core/api.h |   649 +-
> .../drivers/swr/rasterizer/core/arena.h       |   190 +-
> .../drivers/swr/rasterizer/core/backend.cpp   |   281 +-
> .../drivers/swr/rasterizer/core/backend.h     |   106 +-
> .../swr/rasterizer/core/backend_clear.cpp     |   218 +-
> .../swr/rasterizer/core/backend_impl.h        |  1030 +-
> .../swr/rasterizer/core/backend_sample.cpp    |   276 +-
> .../rasterizer/core/backend_singlesample.cpp  |   295 +-
> .../drivers/swr/rasterizer/core/binner.cpp    |  1069 +-
> .../drivers/swr/rasterizer/core/binner.h      |   294 +-
> .../drivers/swr/rasterizer/core/blend.h       |   174 +-
> .../drivers/swr/rasterizer/core/clip.cpp      |   272 +-
> .../drivers/swr/rasterizer/core/clip.h        |   740 +-
> .../swr/rasterizer/core/conservativeRast.h    |   161 +-
> .../drivers/swr/rasterizer/core/context.h     |   386 +-
> .../swr/rasterizer/core/depthstencil.h        |   238 +-
> .../drivers/swr/rasterizer/core/fifo.hpp      |   117 +-
> .../swr/rasterizer/core/format_conversion.h   |   175 +-
> .../swr/rasterizer/core/format_traits.h       |  5102 +++---
> .../swr/rasterizer/core/format_types.h        |   870 +-
> .../swr/rasterizer/core/format_utils.h        |   513 +-
> .../drivers/swr/rasterizer/core/frontend.cpp  |  1033 +-
> .../drivers/swr/rasterizer/core/frontend.h    |   250 +-
> .../drivers/swr/rasterizer/core/knobs.h       |   125 +-
> .../drivers/swr/rasterizer/core/knobs_init.h  |    64 +-
> .../drivers/swr/rasterizer/core/multisample.h |   464 +-
> src/gallium/drivers/swr/rasterizer/core/pa.h  |   798 +-
> .../drivers/swr/rasterizer/core/pa_avx.cpp    |  1504 +-
> .../swr/rasterizer/core/rasterizer.cpp        |   237 +-
> .../drivers/swr/rasterizer/core/rasterizer.h  |   191 +-
> .../swr/rasterizer/core/rasterizer_impl.h     |   779 +-
> .../swr/rasterizer/core/rdtsc_core.cpp        |   175 +-
> .../drivers/swr/rasterizer/core/rdtsc_core.h  |    56 +-
> .../drivers/swr/rasterizer/core/ringbuffer.h  |    89 +-
> .../drivers/swr/rasterizer/core/state.h       |   614 +-
> .../drivers/swr/rasterizer/core/state_funcs.h |    69 +-
> .../drivers/swr/rasterizer/core/tessellator.h |   100 +-
> .../drivers/swr/rasterizer/core/threads.cpp   |   468 +-
> .../drivers/swr/rasterizer/core/threads.h     |   102 +-
> .../drivers/swr/rasterizer/core/tilemgr.cpp   |   301 +-
> .../drivers/swr/rasterizer/core/tilemgr.h     |   205 +-
> .../drivers/swr/rasterizer/core/tileset.h     |    73 +-
> .../drivers/swr/rasterizer/core/utils.h       |   195 +-
> .../swr/rasterizer/jitter/JitManager.cpp      |   426 +-
> .../swr/rasterizer/jitter/JitManager.h        |   140 +-
> .../swr/rasterizer/jitter/blend_jit.cpp       |   336 +-
> .../drivers/swr/rasterizer/jitter/blend_jit.h |   104 +-
> .../drivers/swr/rasterizer/jitter/builder.cpp |    53 +-
> .../drivers/swr/rasterizer/jitter/builder.h   |   217 +-
> .../swr/rasterizer/jitter/builder_gfx_mem.cpp |   137 +-
> .../swr/rasterizer/jitter/builder_gfx_mem.h   |   155 +-
> .../swr/rasterizer/jitter/builder_math.h      |    56 +-
> .../swr/rasterizer/jitter/builder_mem.cpp     |   321 +-
> .../swr/rasterizer/jitter/builder_mem.h       |   216 +-
> .../swr/rasterizer/jitter/builder_misc.cpp    |   482 +-
> .../swr/rasterizer/jitter/builder_misc.h      |   206 +-
> .../swr/rasterizer/jitter/fetch_jit.cpp       |  1494 +-
> .../drivers/swr/rasterizer/jitter/fetch_jit.h |   136 +-
> .../jitter/functionpasses/lower_x86.cpp       |   583 +-
> .../rasterizer/jitter/functionpasses/passes.h |    54 +-
> .../drivers/swr/rasterizer/jitter/jit_api.h   |    73 +-
> .../drivers/swr/rasterizer/jitter/jit_pch.hpp |    74 +-
> .../jitter/shader_lib/DebugOutput.cpp         |    60 +-
> .../swr/rasterizer/jitter/streamout_jit.cpp   |   152 +-
> .../swr/rasterizer/jitter/streamout_jit.h     |    81 +-
> 114 files changed, 27803 insertions(+), 22175 deletions(-)
> 
> -- 
> 2.17.0
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev



More information about the mesa-dev mailing list