HomePhabricator

Merging r339190:

Description

Merging r339190:

r339190 | jvesely | 2018-08-07 23:54:37 +0200 (Tue, 07 Aug 2018) | 12 lines

AMDGPU: Remove broken i16 ternary patterns

Fixup test to check for GCN prefix
These patterns always zero extend the result even though it might need sign extension.
This has been broken since the addition of i16 support.
It has popped up in mad_sat(char) test since min(max()) combination is turned into v_med3, resulting in the following (incorrect) sequence:

v_mad_i16 v2, v10, v9, v11
v_med3_i32 v2, v2, v8, v7

Fixes mad_sat(char) piglit on VI.

Differential Revision: https://reviews.llvm.org/D49836

Details

Committed
hansAug 8 2018, 4:35 AM
Differential Revision
D49836: AMDGPU: Remove broken ternary i16 patterns
Parents
rL339234: Merging r338716:
Branches
Unknown
Tags
Unknown