Re: [qemu-s390x] [Qemu-devel] [PATCH v1 5/5] s390x/tcg: Implement VECTOR STRING RANGE COMPARE

[Richard Henderson]

On 5/15/19 1:31 PM, David Hildenbrand wrote:> +    const bool equal =
extract32(c, BITS - 1, 1);> +    const bool lower = extract32(c, BITS - 2, 1);>
+    const bool higher = extract32(c, BITS - 3, 1);> +> +    if (equal && data
== l) {> +        return true;> +    } else if (lower && data < l) {> +
return true;> +    } else if (higher && data > l) {> +        return true;> +
  }> +    return false;
Only one of the data comparisons can be true.  Better as

    if (data < l) {
        return lower;
    } else if (data > l) {
        return higher;
    } else {
        return equal;
    }

Otherwise, again, let's see if we can turn these into inlines.


r~