|
From: | Richard Henderson |
Subject: | Re: [PATCH v3 13/26] Hexagon (target/hexagon) cleanup ternary operators in semantics |
Date: | Thu, 8 Apr 2021 12:17:49 -0700 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.7.1 |
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~
[Prev in Thread] | Current Thread | [Next in Thread] |