Re: [Qemu-devel] [PULL 00/12] TCG patch queue

[Peter Maydell]

On 6 February 2016 at 00:09, Richard Henderson <address@hidden> wrote:
> This contains two patches from December, and the portion that
> has been reviewed of my tcg-indirect-register series for sparc.
> The latter is mostly cleanup to tcg.c and so is nice to have
> regardless of the rest of the patch set.
>
>
> r~
>
>
> The following changes since commit ee8e8f92a730afc17ab8be6e86df6b9a23b8ebc6:
>
>   Merge remote-tracking branch 
> 'remotes/amit-migration/tags/migration-for-2.6-2' into staging (2016-02-05 
> 14:20:46 +0000)
>
> are available in the git repository at:
>
>   git://github.com/rth7680/qemu.git tags/pull-tcg-20160206
>
> for you to fetch changes up to 78c9f7ee92717671d281882d33ef3f80ca711c95:
>
>   tcg: Introduce temp_load (2016-02-06 10:44:31 +1100)
>
> ----------------------------------------------------------------
> Queued TCG patches.

Hi. This doesn't compile with clang, I'm afraid:

/home/petmay01/linaro/qemu-for-merges/tcg/tcg.c:2048:30: error:
comparison of constant 16 with expression of type 'TCGReg' is always
true [-Werror,-Wtautological-constant-out-of-range-compare]
            for(reg = 0; reg < TCG_TARGET_NB_REGS; reg++) {
                         ~~~ ^ ~~~~~~~~~~~~~~~~~~
/home/petmay01/linaro/qemu-for-merges/tcg/tcg.c:2204:22: error:
comparison of constant 16 with expression of type 'TCGReg' is always
true [-Werror,-Wtautological-constant-out-of-range-compare]
    for(reg = 0; reg < TCG_TARGET_NB_REGS; reg++) {
                 ~~~ ^ ~~~~~~~~~~~~~~~~~~

thanks
-- PMM