[Mesa-dev] [PATCH 00/28] update swr rasterizer
Rowley, Timothy O
timothy.o.rowley at intel.com
Thu Mar 16 19:01:16 UTC 2017
On Mar 15, 2017, at 9:59 PM, Emil Velikov <emil.l.velikov at gmail.com<mailto:emil.l.velikov at gmail.com>> wrote:
Hi Tim,
On 16 March 2017 at 00:12, Tim Rowley <timothy.o.rowley at intel.com<mailto:timothy.o.rowley at intel.com>> wrote:
Hightlights include: lots of simd16 work, assert rework, and autogen
changes (scripts centralized, one file added, two removed).
Tim Rowley (28):
swr: [rasterizer core] Support sparse numa id values on all OSes
swr: [rasterizer core] Finish SIMD16 PA OPT except tesselation
swr: [rasterizer core] Finish SIMD16 PA OPT including tesselation
swr: [rasterizer core/scripts] Autogen backend initialization
function(s)
swr: [rasterizer archrast] Add additional API events
swr: [rasterizer core] Implement SIMD16 GS and STREAMOUT
swr: [rasterizer archrast] Fix performance issue with archrast stats
swr: [rasterizer common] Add InterpolateComponentFlat utility
swr: [rasterizer core] Fix RECT_LIST primitive assembly
swr: [rasterizer archrast/scripts] Further archrast cleanups
swr: [rasterizer archrast] Remove redundant data from archrast files
swr: [rasterizer archrast/core/scripts] Fix archrast multithreading
issue
swr: [rasterizer core] Implement double pumped SIMD16 TESS
swr: [rasterizer archrast] Fix the early and late depthstencil events
swr: [rasterizer] Backend code adjustments
swr: [rasterizer] Slight assert refactoring
swr: [rasterizer core] Allow no arguments to SWR_INVALID macro
swr: [rasterizer core/common] Fix the native AVX512 build under ICC
swr: [rasterizer core] Fix typo in SIMD16 code path
swr: [rasterizer] Convert more SWR_ASSERT(false, ...) to
SWR_INVALID(...)
swr: [rasterizer jitter] Fix LogicOp blend jit after assert changes
swr: [rasterizer core] SIMD16 Frontend WIP - fix tesselation crashes
swr: [rasterizer core] fix trifan regression from 52f9f54dce
swr: [rasterizer scripts] Put codegen scripts into a separate
directory
swr: [rasterizer codegen] Quiet gen_backends.py execution
swr: [rasterizer codegen] Rewrite gen_llvm_ir_macros.py to use mako
swr: [rasterizer codegen] Fix generation of knobs
swr: [rasterizer codegen] Rewrite gen_llvm_types.py to use mako
Above all - thanks for omitting the execute bit/shebang for the new
python script !
A few high-level comments/suggestions.
v2 of the patch upcoming with the following addressed as follows.
- seems like this series will break the scons build
You really do _not_ want to do that, please ensure that things build
[always] with both scons and autoconf.
Ah, I keep forgetting about the second build system. I’ve gone back and added the scons changes.
- there's at least one patch that mixes how licence text is commented
alongside other changes
Please keep that to a separate patch ?
Separated out the comment style change patch.
- some variables (gBackendPixelRateTable comes to mind) are declared
in various places, and during the series not all instances are updated
at once
Just move them to a header, in a prep patch ?
Moved declaration of gBackendPixelRateTable to backend.h in a patch prior to the autogen and other changes.
- renaming actual mako templates to .cpp/.hpp is confusing/misleading
Worth leaving as-is and/or checking how other generators in-tree name
such files ?
The idea behind using a cpp/hpp extension was that the editor would better automatically handle the template files during editing. In the interest of consistency, a commit has been added to cleanup the naming so all template and generated files are now prefixed with gen_.
I would strongly urge you to address the first issue, but at the end
of the day it's wishful thinking.
Thanks
Emil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170316/ef643340/attachment-0001.html>
More information about the mesa-dev
mailing list