URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=79d403417cacd2728916e32ae55f4fc2a018515c
Author: Jason Ekstrand <[email protected]>
Date: Fri Sep 29 12:22:48 2017 -0700
intel/cs: Make thread_local_id a regular builtin param
This is a lot more natural than special casing it all over the place.
We still have to do a bit of special-casing in assign_constant_locations
but it's not special-cased quite as bad as it was before.
Reviewed-by: Jordan Justen <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=8d90e2883954eb7022cf8fc98be3773cc5513e7b
Author: Jason Ekstrand <[email protected]>
Date: Fri Sep 29 11:30:25 2017 -0700
intel/compiler: Allocate pull_param in assign_constant_locations
Now that everything is nicely ralloc'd, we can allocate the pull_param
array in assign_constant_locations instead of higher up. We can also
re-allocate the param array so that it's exactly the needed size. This
should save us some memory because we're not allocating the total needed
param space for both push and pull.
Reviewed-by: Jordan Justen <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=29737eac985cf028b19d977cb8fa0d7320cf91cf
Author: Jason Ekstrand <[email protected]>
Date: Thu Sep 28 21:45:41 2017 -0700
intel: Allocate prog_data::[pull_]param deeper inside the compiler
Now that we're always growing the param array as-needed, we can
allocate the param array in common code and stop repeating the
allocation everywere. In order to keep things sane, we ralloc the
[pull_]param array off of the compile context and then steal it back
to a NULL context later. This doesn't get us all the way to where
prog_data::[pull_]param is purely an out parameter of the back-end
compiler but it gets us a lot closer.
Reviewed-by: Jordan Justen <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=c3d54d03757fcb656cc4839a2c7978d97f75508d
Author: Jason Ekstrand <[email protected]>
Date: Wed Aug 16 16:50:46 2017 -0700
ralloc: Allow reparenting to a NULL context
Reviewed-by: Tapani Pälli <[email protected]>
Reviewed-by: Ian Romanick <[email protected]>
Reviewed-by: Jordan Justen <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=2e317a4b6d77ab078bdc9b21765f6051c4577c5e
Author: Jason Ekstrand <[email protected]>
Date: Fri Sep 29 11:18:04 2017 -0700
anv/pipeline: Refactor setup of the prog_data::param array
Now that the only thing we put in the array up-front are client push
constants, we can simplify anv_pipeline_compile a bit.
Reviewed-by: Jordan Justen <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=6b31229592df76eefc1aa41d5b936ad2bb6b5598
Author: Jason Ekstrand <[email protected]>
Date: Fri Sep 29 11:10:22 2017 -0700
anv/pipeline: Grow the param array for images
Before, we were calculating up-front and then filling in later. Now we
just grow as needed in anv_nir_apply_pipeline_layout.
Reviewed-by: Jordan Justen <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=63c938fd1838d372674db70b7ff0199594a85cd0
Author: Jason Ekstrand <[email protected]>
Date: Fri Sep 29 11:09:04 2017 -0700
anv/pipeline: Whack nir->num_uniforms to MAX_PUSH_CONSTANT_SIZE
This way any image uniforms end up having locations higher than
MAX_PUSH_CONSTANT_SIZE. There's no bug here at the moment, but this
consistency will make the next commit easier. Also, because
nir_apply_pipeline_layout properly increments nir->num_uniforms when
it expands the param array, we no longer need to stomp it to match
prog_data::nr_params because it already does.
Reviewed-by: Jordan Justen <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=4dfb8b3416e2c7c069ac11736362e9dc3860f72c
Author: Jason Ekstrand <[email protected]>
Date: Fri Sep 29 10:50:07 2017 -0700
intel/vs: Grow the param array for clip planes
Instead of requiring the caller of brw_compile_vs to figure it out, just
grow the param array on-demand.
Reviewed-by: Jordan Justen <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=6bcc5c0c75226bb89f35d7529de11182051c729e
Author: Jason Ekstrand <[email protected]>
Date: Fri Sep 29 10:37:40 2017 -0700
intel/cs: Grow prog_data::param on-demand for thread_local_id_index
Instead of making the caller of brw_compile_cs add something to the
param array for thread_local_id_index, just add it on-demand in
brw_nir_intrinsics and grow the array. This is now safe to do because
everyone is now using ralloc for prog_data::param.
Reviewed-by: Jordan Justen <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=b1d1b7222acfe45e447d1708d60b3ff98ca5a9a7
Author: Jason Ekstrand <[email protected]>
Date: Fri Oct 6 10:08:11 2017 -0700
intel/compiler: Make brw_nir_lower_intrinsics compute-specific
It's already only ever called from brw_compile_cs and only handles
compute intrinsics. Let's just make it CS-specific. We can always
make it handle other stages again later if we want.
Reviewed-by: Jordan Justen <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=2db9470d88bb4856152465f44fa090f1a1bbf039
Author: Jason Ekstrand <[email protected]>
Date: Fri Sep 29 11:05:55 2017 -0700
intel/compiler: Add a helper for growing the prog_data::param array
Reviewed-by: Jordan Justen <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=c0435b204a51ae2d3409416fe11122f3b7c88bc3
Author: Jason Ekstrand <[email protected]>
Date: Fri Sep 29 10:12:12 2017 -0700
intel/compiler: Stop adding params for texture sizes
We haven't needed this ever since we started using NIR for lowering
rectangle textures.
Reviewed-by: Jordan Justen <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=4d4f14937683110c7b61acb1555f31d070316b04
Author: Jason Ekstrand <[email protected]>
Date: Fri Sep 29 10:16:43 2017 -0700
i965: Only add the wpos state reference if we lowered something
Otherwise, in the ARB program case _mesa_add_state_reference may grow
the parameter array which will cause brw_nir_setup_arb_uniforms to write
past the end of the param array because it only looks at the parameter
list length but the parma array is allocated based on nir->num_uniforms.
The only reason this hasn't caused us problems is because we are padding
out the param array for fragment programs unnecessarily.
Reviewed-by: Jordan Justen <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=4efd079aba165d5a046868ec77a27605724da858
Author: Jason Ekstrand <[email protected]>
Date: Thu Sep 28 22:16:55 2017 -0700
intel/compiler: Add a flag for pull constant support
The Vulkan driver does not support pull constants. It simply limits
things such that we can always push everything. Previously, we were
determining whether or not to push things based on whether or not the
prog_data::pull_param array is non-null. This is rather hackish and
about to stop working.
Reviewed-by: Jordan Justen <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=9df64b56663c3ffaee081aa608db9e5163a4eeae
Author: Jason Ekstrand <[email protected]>
Date: Fri Sep 29 10:06:17 2017 -0700
anv/pipeline: Ralloc prog_data::param of the compile mem_ctx
This way we stop leaking it. This is completely safe because, when we
hand it off to anv_shader_bin_create or anv_pipeline_cache_upload_kernel,
they make a copy of the entire param array.
Reviewed-by: Jordan Justen <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=490d80fd1a55287b31ce62d0bf07801316632e78
Author: Jason Ekstrand <[email protected]>
Date: Thu Sep 28 21:51:48 2017 -0700
anv/pipeline: Add a mem_ctx parameter to anv_pipeline_compile
This lets us avoid some of the manual ralloc stealing and prepares for
future commits in which we will want to ralloc prog_data::param.
Reviewed-by: Jordan Justen <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=cfc7ed75eb8baab69308ed520f97a3efd75fb548
Author: Jason Ekstrand <[email protected]>
Date: Thu Sep 28 19:04:31 2017 -0700
i965: Store image_param in brw_context instead of prog_data
This burns an extra 10k of memory or so in the case where you don't have
any images. However, if you have several shaders which use images, this
should be much less memory. It also gets rid of a part of prog_data
that really has nothing to do with the compiler.
Reviewed-by: Jordan Justen <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=6ee4b352c90e93c82d900328716c1068a7188b91
Author: Jason Ekstrand <[email protected]>
Date: Thu Sep 28 19:03:38 2017 -0700
i965: Use prog->info.num_images for needs_dc computation
This should be just as good as looking in prog_data but removes our one
state setup dependency on brw_stage_prog_data::nr_image_param.
Reviewed-by: Jordan Justen <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=2975e4c56a7aeade5a324aa4d446f18cc176fa06
Author: Jason Ekstrand <[email protected]>
Date: Thu Sep 28 16:25:31 2017 -0700
intel: Rewrite the world of push/pull params
This moves us away to the array of pointers model and onto a model where
each param is represented by a generic uint32_t handle. We reserve 2^16
of these handles for builtins that get generated by somewhere inside the
compiler and have well-defined meanings. Generic params have handles
whose meanings are defined by the driver.
The primary downside to this new approach is that it moves a little bit
of the work that we would normally do at compile time to draw time. On
my laptop this hurts OglBatch6 by no more than 1% and doesn't seem to
have any measurable affect on OglBatch7. So, while this may come back
to bite us, it doesn't look too bad.
Reviewed-by: Jordan Justen <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=faad828b16448c1008a1b15ac8d8a72b13005c09
Author: Jason Ekstrand <[email protected]>
Date: Thu Sep 28 15:27:46 2017 -0700
i965: Get rid of gen7_cs_state.c
The only thing it was handling was push constants. We pull the actual
constant upload code into gen6_constant_state.c and the atoms into
genX_state_upload.c.
Reviewed-by: Jordan Justen <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=9b3f917f9ed177e6de6fc7231dc9a0ab75301dac
Author: Jason Ekstrand <[email protected]>
Date: Thu Sep 28 15:05:51 2017 -0700
i965: Add a helper for populating constant buffers
Reviewed-by: Jordan Justen <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=d6406271595f9a3cb765cf2be76ff5c2bf0cf8d7
Author: Jason Ekstrand <[email protected]>
Date: Thu Sep 28 14:39:00 2017 -0700
i965: Move brw_upload_pull_constants to gen6_constant_state.c
Reviewed-by: Jordan Justen <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=3442c9fc3ebd5de2c9d6c0b9ce541f182809fe82
Author: Jason Ekstrand <[email protected]>
Date: Mon Aug 28 17:06:24 2017 -0700
nir: Get rid of the variable on vote intrinsics
This looks like a copy+paste error. They don't actually write into that
variable as would be implied by putting the return there.
Reviewed-by: Lionel Landwerlin <[email protected]>
Cc: [email protected]
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=a0947921eb01f4c037de28b753fc10f86a25fc65
Author: Jason Ekstrand <[email protected]>
Date: Mon Aug 28 15:05:11 2017 -0700
nir/opcodes: Fix constant-folding of ufind_msb
We didn't fold correctly in the case of 0x1 because we never let the
loop counter hit 0. Switching it to bit >= 0 solves this problem.
Reviewed-by: Bas Nieuwenhuizen <[email protected]>
Reviewed-by: Connor Abbott <[email protected]>
Cc: [email protected]
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=ac3b73ac8dcf501ab704b274d67acd065fbd4348
Author: Jason Ekstrand <[email protected]>
Date: Thu Oct 12 22:36:48 2017 -0700
meta: Delete the PBO texsubimage path for real
Acked-by: Kenneth Graunke <[email protected]>
_______________________________________________
mesa-commit mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-commit
