[PATCH 1/1] Support: Use const pointers for reads.

Rui Ueyama ruiu at google.com
Mon Mar 2 13:48:22 PST 2015


LGTM.

On Mon, Mar 2, 2015 at 1:44 PM, Jan Vesely <jan.vesely at rutgers.edu> wrote:

> Fixes tons of const-cast warnings.
>
> Signed-off-by: Jan Vesely <jan.vesely at rutgers.edu>
> ---
>  include/llvm/Support/Endian.h | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/include/llvm/Support/Endian.h b/include/llvm/Support/Endian.h
> index db85c65..17ae651 100644
> --- a/include/llvm/Support/Endian.h
> +++ b/include/llvm/Support/Endian.h
> @@ -197,12 +197,12 @@ typedef detail::packed_endian_specific_integral
>                     <int64_t, native, unaligned> unaligned_int64_t;
>
>  namespace endian {
> -inline uint16_t read16le(const void *p) { return *(ulittle16_t *)p; }
> -inline uint32_t read32le(const void *p) { return *(ulittle32_t *)p; }
> -inline uint64_t read64le(const void *p) { return *(ulittle64_t *)p; }
> -inline uint16_t read16be(const void *p) { return *(ubig16_t *)p; }
> -inline uint32_t read32be(const void *p) { return *(ubig32_t *)p; }
> -inline uint64_t read64be(const void *p) { return *(ubig64_t *)p; }
> +inline uint16_t read16le(const void *p) { return *(const ulittle16_t *)p;
> }
> +inline uint32_t read32le(const void *p) { return *(const ulittle32_t *)p;
> }
> +inline uint64_t read64le(const void *p) { return *(const ulittle64_t *)p;
> }
> +inline uint16_t read16be(const void *p) { return *(const ubig16_t *)p; }
> +inline uint32_t read32be(const void *p) { return *(const ubig32_t *)p; }
> +inline uint64_t read64be(const void *p) { return *(const ubig64_t *)p; }
>
>  inline void write16le(void *p, uint16_t v) { *(ulittle16_t *)p = v; }
>  inline void write32le(void *p, uint32_t v) { *(ulittle32_t *)p = v; }
> --
> 2.1.0
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150302/06aa5c54/attachment.html>


More information about the llvm-commits mailing list