diff options
| author | Daniel Schürmann <[email protected]> | 2020-06-12 11:59:27 +0100 |
|---|---|---|
| committer | Marge Bot <[email protected]> | 2020-06-12 13:56:12 +0000 |
| commit | 1f98d8c804094fcda1c607234433f2ba2be2cf07 (patch) | |
| tree | 3a7c334f878d012e8ec13a02de2e8b6354c87c8f /src | |
| parent | e7e9969efec777443a022056939049dfa82c5cf4 (diff) | |
aco: fix shared subdword loads
Shared subdword loads don't need byte alignment as they are split
into multiple loads if necessary.
Fixes: 5cde4989d3c8c25b0ba2a11ec450625e30092b16 ('aco: remove unnecessary split- and create_vector instructions for subdword loads')
Reviewed-by: Rhys Perry <[email protected]>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/5441>
Diffstat (limited to 'src')
| -rw-r--r-- | src/amd/compiler/aco_instruction_selection.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/amd/compiler/aco_instruction_selection.cpp b/src/amd/compiler/aco_instruction_selection.cpp index ffe5f2e4619..38bf449c83b 100644 --- a/src/amd/compiler/aco_instruction_selection.cpp +++ b/src/amd/compiler/aco_instruction_selection.cpp @@ -3192,7 +3192,7 @@ void emit_load(isel_context *ctx, Builder& bld, const LoadEmitInfo *info) } /* shift result right if needed */ - if (info->component_size < 4) { + if (info->component_size < 4 && byte_align_loads) { Operand align((uint32_t)byte_align); if (byte_align == -1) { if (offset.isConstant()) |