diff options
| author | Eric Anholt <[email protected]> | 2017-11-01 15:16:59 -0700 |
|---|---|---|
| committer | Eric Anholt <[email protected]> | 2017-11-07 09:19:48 -0800 |
| commit | 078b163a9c581e5b0e7bcc7436f0bf569cbc020d (patch) | |
| tree | dadb992d438dec013473a147831be2a021d5d9cb /src/gallium/drivers/vc5/vc5_cl.h | |
| parent | 735b844b1b32a440738b1a6096ea3aa4df6b52a7 (diff) | |
broadcom/vc5: Add a cl_emit() variant for merging with a pre-packed struct.
Cleans up the hand-written code, at the cost of another ugly macro.
Diffstat (limited to 'src/gallium/drivers/vc5/vc5_cl.h')
| -rw-r--r-- | src/gallium/drivers/vc5/vc5_cl.h | 19 |
1 files changed, 17 insertions, 2 deletions
diff --git a/src/gallium/drivers/vc5/vc5_cl.h b/src/gallium/drivers/vc5/vc5_cl.h index 64ccac80593..4c64f084d3d 100644 --- a/src/gallium/drivers/vc5/vc5_cl.h +++ b/src/gallium/drivers/vc5/vc5_cl.h @@ -221,8 +221,23 @@ cl_get_emit_space(struct vc5_cl_out **cl, size_t size) ({ \ struct vc5_cl_out *cl_out = cl_start(cl); \ cl_packet_pack(packet)(cl, (uint8_t *)cl_out, &name); \ - VG(VALGRIND_CHECK_MEM_IS_DEFINED(cl_out, \ - cl_packet_length(packet))); \ + cl_advance(&cl_out, cl_packet_length(packet)); \ + cl_end(cl, cl_out); \ + _loop_terminate = NULL; \ + })) \ + +#define cl_emit_with_prepacked(cl, packet, prepacked, name) \ + for (struct cl_packet_struct(packet) name = { \ + cl_packet_header(packet) \ + }, \ + *_loop_terminate = &name; \ + __builtin_expect(_loop_terminate != NULL, 1); \ + ({ \ + struct vc5_cl_out *cl_out = cl_start(cl); \ + uint8_t packed[cl_packet_length(packet)]; \ + cl_packet_pack(packet)(cl, packed, &name); \ + for (int _i = 0; _i < cl_packet_length(packet); _i++) \ + ((uint8_t *)cl_out)[_i] = packed[_i] | (prepacked)[_i]; \ cl_advance(&cl_out, cl_packet_length(packet)); \ cl_end(cl, cl_out); \ _loop_terminate = NULL; \ |