| commit | 5ed76a4c63db9295c6c5d67895925810050d4a46 | |
| author | Paolo Bonzini <pbonzini@redhat.com> | |
| Mon, 4 Feb 2019 15:40:18 +0000 (4 16:40 +0100) | ||
| committer | Paolo Bonzini <pbonzini@redhat.com> | |
| Tue, 5 Feb 2019 15:50:21 +0000 (5 16:50 +0100) | ||
| tree | f5f95ef0d7804f1a93c6817e6853790cc82be3e2 | treesnapshot (tar.gz zip) |
| parent | 568534986486e619258c6ff36b4029372624238a | commitdiff |
queue: fix QTAILQ_FOREACH_REVERSE_SAFE
The iteration was stopping as soon as prev_var was set to NULL, and
therefore it skipped the first element. Fortunately, or unfortunately,
we have only one use of QTAILQ_FOREACH_REVERSE_SAFE. Thus this only
showed up as incorrect register preferences on the very first translation
block that was compiled.
Reported-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Emilio G. Cota <cota@braap.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
The iteration was stopping as soon as prev_var was set to NULL, and
therefore it skipped the first element. Fortunately, or unfortunately,
we have only one use of QTAILQ_FOREACH_REVERSE_SAFE. Thus this only
showed up as incorrect register preferences on the very first translation
block that was compiled.
Reported-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Emilio G. Cota <cota@braap.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
| include/qemu/queue.h | diffblobblamehistory |