Re: [PATCH v3 13/26] Hexagon (target/hexagon) cleanup ternary operators in semantics

[Richard Henderson]

On 4/7/21 6:57 PM, Taylor Simpson wrote:
> Change  (cond ? (res = x) : (res = y)) to res = (cond ? x : y)
>
> This makes the semnatics easier to for idef-parser to deal with
>
> The following instructions are impacted
>      C2_any8
>      C2_all8
>      C2_mux
>      C2_muxii
>      C2_muxir
>      C2_muxri
>
> Signed-off-by: Taylor Simpson<tsimpson@quicinc.com>
> ---
>   target/hexagon/imported/compare.idef | 12 ++++++------
>   1 file changed, 6 insertions(+), 6 deletions(-)

I presume this change has also been made in the upstream Qualcomm source, so 
that the next import does not revert it?  Anyway,

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~