Re: suggestion for Merging LLVM

[Christopher Li <sparse@xxxxxxxxxxx>]

On Tue, Dec 13, 2011 at 12:29 PM, Pekka Enberg <penberg@xxxxxxxxxx> wrote:
>
> Ping? LLVM 3.0 is out now officially.


Glad to heard that LLVM 3.0 is out. I haven't able to make the llvm
2.x work yet.
I am fine with LLVM 3.0 for now.

I have one question regarding this change:
2ded1e7406914eda77abde035416140849d76f68

    sparse: Bump up sizeof(_Bool) to 8 bits

    We need sizeof(_Bool) to be one byte to generate code for boolean
expressions
    in the LLVM backend.

Sparse already evaluate sizeof(_Bool) as 8 bits. It current just give warnings.
Will remove the warning sufficient to make sparse-llvm happy?
I take a look at the code LLVM generate, It seem LLVM treat _Bool as 1
bit integer
as well. Can we not change the internal representation of _Bool, only
the sizeof(_Bool)?

Thanks

Chris
--
To unsubscribe from this list: send the line "unsubscribe linux-sparse" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html