<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">
<br class="">
<div>
<blockquote type="cite" class="">
<div class="">On Jun 26, 2017, at 7:57 AM, Emil Velikov <<a href="mailto:emil.l.velikov@gmail.com" class="">emil.l.velikov@gmail.com</a>> wrote:</div>
<br class="Apple-interchange-newline">
<div class=""><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">Hi
 Tim,</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">
<br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">
<span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">On
 22 June 2017 at 22:13, Tim Rowley <</span><a href="mailto:timothy.o.rowley@intel.com" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;" class="">timothy.o.rowley@intel.com</a><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">>
 wrote:</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">
<blockquote type="cite" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;" class="">
Hardcode split to four files currently.  Decreases swr build<br class="">
time on a quad-core by ~10%.<br class="">
---<br class="">
src/gallium/drivers/swr/Makefile.am                |   26 +-<br class="">
src/gallium/drivers/swr/Makefile.sources           |    4 +<br class="">
src/gallium/drivers/swr/SConscript                 |   19 +-<br class="">
.../drivers/swr/rasterizer/codegen/gen_backends.py |   38 +-<br class="">
.../drivers/swr/rasterizer/codegen/gen_common.py   |    7 +<br class="">
.../rasterizer/codegen/templates/gen_backend.cpp   |    1 +<br class="">
.../codegen/templates/gen_header_init.hpp          |   43 +<br class="">
src/gallium/drivers/swr/rasterizer/core/api.cpp    |    7 +-<br class="">
.../drivers/swr/rasterizer/core/backend.cpp        |  809 +--------------<br class="">
src/gallium/drivers/swr/rasterizer/core/backend.h  | 1033 +------------------<br class="">
.../drivers/swr/rasterizer/core/backend_clear.cpp  |  281 ++++++<br class="">
.../drivers/swr/rasterizer/core/backend_impl.h     | 1067 ++++++++++++++++++++<br class="">
.../drivers/swr/rasterizer/core/backend_sample.cpp |  345 +++++++<br class="">
.../swr/rasterizer/core/backend_singlesample.cpp   |  321 ++++++<br class="">
14 files changed, 2160 insertions(+), 1841 deletions(-)<br class="">
create mode 100644 src/gallium/drivers/swr/rasterizer/codegen/templates/gen_header_init.hpp<br class="">
create mode 100644 src/gallium/drivers/swr/rasterizer/core/backend_clear.cpp<br class="">
create mode 100644 src/gallium/drivers/swr/rasterizer/core/backend_impl.h<br class="">
create mode 100644 src/gallium/drivers/swr/rasterizer/core/backend_sample.cpp<br class="">
create mode 100644 src/gallium/drivers/swr/rasterizer/core/backend_singlesample.cpp<br class="">
<br class="">
diff --git a/src/gallium/drivers/swr/Makefile.am b/src/gallium/drivers/swr/Makefile.am<br class="">
index 6650abd..0daec90 100644<br class="">
--- a/src/gallium/drivers/swr/Makefile.am<br class="">
+++ b/src/gallium/drivers/swr/Makefile.am<br class="">
@@ -34,6 +34,7 @@ COMMON_CXXFLAGS = \<br class="">
       $(LLVM_CXXFLAGS) \<br class="">
       $(SWR_CXX11_CXXFLAGS) \<br class="">
       -I$(builddir)/rasterizer/codegen \<br class="">
+       -I$(builddir)/rasterizer/core \<br class="">
       -I$(builddir)/rasterizer/jitter \<br class="">
       -I$(builddir)/rasterizer/archrast \<br class="">
       -I$(srcdir)/rasterizer \<br class="">
@@ -62,7 +63,11 @@ BUILT_SOURCES = \<br class="">
       rasterizer/archrast/gen_ar_event.cpp \<br class="">
       rasterizer/archrast/gen_ar_eventhandler.hpp \<br class="">
       rasterizer/archrast/gen_ar_eventhandlerfile.hpp \<br class="">
-       rasterizer/core/gen_BackendPixelRate0.cpp<br class="">
+       rasterizer/core/backends/gen_BackendPixelRate0.cpp \<br class="">
+       rasterizer/core/backends/gen_BackendPixelRate1.cpp \<br class="">
+       rasterizer/core/backends/gen_BackendPixelRate2.cpp \<br class="">
+       rasterizer/core/backends/gen_BackendPixelRate3.cpp \<br class="">
+       rasterizer/core/backends/gen_BackendPixelRate.hpp<br class="">
<br class="">
MKDIR_GEN = $(AM_V_at)$(MKDIR_P) $(@D)<br class="">
PYTHON_GEN = $(AM_V_GEN)$(PYTHON2) $(PYTHON_FLAGS)<br class="">
@@ -140,20 +145,30 @@ rasterizer/archrast/gen_ar_eventhandlerfile.hpp: rasterizer/codegen/gen_archrast<br class="">
               --output rasterizer/archrast/gen_ar_eventhandlerfile.hpp \<br class="">
               --gen_eventhandlerfile_h<br class="">
<br class="">
+rasterizer/core/backends/gen_BackendPixelRate0.cpp \<br class="">
+rasterizer/core/backends/gen_BackendPixelRate1.cpp \<br class="">
+rasterizer/core/backends/gen_BackendPixelRate2.cpp \<br class="">
+rasterizer/core/backends/gen_BackendPixelRate3.cpp \<br class="">
+rasterizer/core/backends/gen_BackendPixelRate.hpp: \<br class="">
+backend.intermediate<br class="">
+<br class="">
# 5 SWR_MULTISAMPLE_TYPE_COUNT<br class="">
# 2 SWR_MSAA_SAMPLE_PATTERN_COUNT<br class="">
# 3 SWR_INPUT_COVERAGE_COUNT<br class="">
# 2 centroid<br class="">
# 2 forcedSampleCount<br class="">
# 2 canEarlyZ<br class="">
-rasterizer/core/gen_BackendPixelRate0.cpp: rasterizer/codegen/gen_backends.py rasterizer/codegen/templates/gen_backend.cpp<br class="">
+<br class="">
+.INTERMEDIATE: backend.intermediate<br class="">
</blockquote>
<span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">I
 have limited experience with .INTERMEDIATE and it didn't seem to</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">
<span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">bring
 single/incremental build times improvements.</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">
<span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">Have
 you seen any on your end? If not I'll just drop it.</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">
</div>
</blockquote>
<div><br class="">
</div>
<div>I’m not really familiar with .INTERMEDIATE myself; found it when googling around looking for a way to specify a code generator rule that produced multiple files.  If there’s a better/cleaner way of doing this I’d like to hear about it.</div>
<br class="">
<blockquote type="cite" class="">
<div class=""><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">
<blockquote type="cite" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;" class="">
+backend.intermediate: rasterizer/codegen/gen_backends.py rasterizer/codegen/templates/gen_backend.cpp rasterizer/codegen/templates/gen_header_init.hpp<br class="">
       $(MKDIR_GEN)<br class="">
       $(PYTHON_GEN) \<br class="">
               $(srcdir)/rasterizer/codegen/gen_backends.py \<br class="">
-               --outdir rasterizer/core \<br class="">
+               --outdir rasterizer/core/backends \<br class="">
               --dim 5 2 3 2 2 2 \<br class="">
-               --split 0 \<br class="">
-               --cpp<br class="">
+               --numfiles 4 \<br class="">
+               --cpp \<br class="">
+               --hpp<br class="">
<br class="">
</blockquote>
<span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">Hardcoding
 file names in generator scripts tends to be a bad idea. One</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">
<span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">example
 is the extra code needed to generate the cmake bits :-)</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">
<span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">One
 could prune that, but it's not a priority AFAICT.</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">
</div>
</blockquote>
<div><br class="">
</div>
<div>I would like to be able to wildcard on the generated name, but it seems that automake wants to have a static list of filenames at invocation.  Our cmake approach internally generates a cmake fragment that is included by the parent cmake, which is a little
 confusing but adds flexibility.</div>
<br class="">
<blockquote type="cite" class="">
<div class=""><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">
<br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">
<blockquote type="cite" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;" class="">
--- a/src/gallium/drivers/swr/rasterizer/codegen/gen_backends.py<br class="">
+++ b/src/gallium/drivers/swr/rasterizer/codegen/gen_backends.py<br class="">
@@ -1,7 +1,7 @@<br class="">
# Copyright (C) 2017 Intel Corporation.   All Rights Reserved.<br class="">
#<br class="">
# Permission is hereby granted, free of charge, to any person obtaining a<br class="">
-# copy of this software and associated documentation files (the "Software"),<br class="">
+# copy of this software and associated documentation files (the 'Software'),<br class="">
# to deal in the Software without restriction, including without limitation<br class="">
# the rights to use, copy, modify, merge, publish, distribute, sublicense,<br class="">
# and/or sell copies of the Software, and to permit persons to whom the<br class="">
@@ -11,7 +11,7 @@<br class="">
# paragraph) shall be included in all copies or substantial portions of the<br class="">
# Software.<br class="">
#<br class="">
-# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR<br class="">
+# THE SOFTWARE IS PROVIDED 'AS IS', WITHOUT WARRANTY OF ANY KIND, EXPRESS OR<br class="">
# IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,<br class="">
# FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL<br class="">
# THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER<br class="">
@@ -31,23 +31,28 @@ from gen_common import ArgumentParser, MakoTemplateWriter<br class="">
<br class="">
def main(args=sys.argv[1:]):<br class="">
    thisDir = os.path.dirname(os.path.realpath(__file__))<br class="">
-    parser = ArgumentParser("Generate files and initialization functions for all permutuations of BackendPixelRate.")<br class="">
-    parser.add_argument('--dim', help="gBackendPixelRateTable array dimensions", nargs='+', type=int, required=True)<br class="">
-    parser.add_argument('--outdir', help="output directory", nargs='?', type=str, default=thisDir)<br class="">
-    parser.add_argument('--split', help="how many lines of initialization per file [0=no split]", nargs='?', type=int, default='512')<br class="">
-    parser.add_argument('--cpp', help="Generate cpp file(s)", action='store_true', default=False)<br class="">
-    parser.add_argument('--cmake', help="Generate cmake file", action='store_true', default=False)<br class="">
+    parser = ArgumentParser('Generate files and initialization functions for all permutuations of BackendPixelRate.')<br class="">
+    parser.add_argument('--dim', help='gBackendPixelRateTable array dimensions', nargs='+', type=int, required=True)<br class="">
+    parser.add_argument('--outdir', help='output directory', nargs='?', type=str, default=thisDir)<br class="">
+    parser.add_argument('--split', help='how many lines of initialization per file [0=no split]', nargs='?', type=int, default='512')<br class="">
+    parser.add_argument('--numfiles', help='how many output files to generate', nargs='?', type=int, default='0')<br class="">
+    parser.add_argument('--cpp', help='Generate cpp file(s)', action='store_true', default=False)<br class="">
+    parser.add_argument('--hpp', help='Generate hpp file', action='store_true', default=False)<br class="">
+    parser.add_argument('--cmake', help='Generate cmake file', action='store_true', default=False)<br class="">
<br class="">
</blockquote>
<span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">Most
 of the above seem unrelated cleanups/fixes. Please keep them separate.</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">
<br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">
<br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">
<blockquote type="cite" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;" class="">
@@ -95,6 +102,19 @@ def main(args=sys.argv[1:]):<br class="">
                fileNum=fileNum,<br class="">
                funcList=chunkedList[fileNum])<br class="">
<br class="">
+    if args.hpp:<br class="">
+        baseHppName = os.path.join(args.outdir, backend.outHeaderName)<br class="">
+        templateHpp = os.path.join(thisDir, 'templates', backend.hpp_template)<br class="">
+<br class="">
+        #print('Generating', filename)<br class="">
</blockquote>
<span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">[snip]</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">
<br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">
<blockquote type="cite" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;" class="">
    # generate gen_backend.cmake file<br class="">
    if args.cmake:<br class="">
        templateCmake = os.path.join(thisDir, 'templates', 'gen_backend.cmake')<br class="">
@@ -108,7 +128,7 @@ def main(args=sys.argv[1:]):<br class="">
            numFiles=numFiles,<br class="">
            baseCppName='${RASTY_GEN_SRC_DIR}/backends/' + os.path.basename(baseCppName))<br class="">
<br class="">
-    #print("Generated %d template instantiations in %d files" % (len(output_list), numFiles))<br class="">
+    #print('Generated %d template instantiations in %d files' % (len(output_list), numFiles))<br class="">
<br class="">
</blockquote>
<span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">Uncomment
 these prints or drop them? They don't seem to bring much as-is.</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">
<br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">
</div>
</blockquote>
<div><br class="">
</div>
<div>I’ll see about separating out the cleanups (including removing commented debug prints) into a separate changelist.</div>
<br class="">
<blockquote type="cite" class="">
<div class=""><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">With
 the .INTERMEDIATE commente (2-3 lines just above it) or dropped,</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">
<span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">the
 build bits are</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">
<span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">Reviewed-by:
 Emil Velikov <</span><a href="mailto:emil.velikov@collabora.com" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;" class="">emil.velikov@collabora.com</a><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">></span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">
<br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">
<span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">-Emil</span></div>
</blockquote>
</div>
<br class="">
</body>
</html>