Re: [RFC PATCH v7 09/29] target/loongarch: Add TLB instruction support

[Richard Henderson]

On 3/28/22 06:57, Xiaojuan Yang wrote:
> +void helper_tlbflush(CPULoongArchState *env)
> +{
> +    int i, index;
> +
> +    index = FIELD_EX64(env->CSR_TLBIDX, CSR_TLBIDX, INDEX);
> +
> +    if (index < LOONGARCH_STLB) {
> +        /* STLB. One line per operation */
> +        for (i = 0; i < 8; i++) {
> +            int index = i * 256 + (index % 256);

Another uninitialized use Werror that you should have seen: You're shadowing the outer 
'index' variable, which means this doesn't do what you intended.


r~