[PATCH] [PR96608] analyzer: Change cast from long to intptr_t
David Malcolm
dmalcolm@redhat.com
Tue Oct 27 14:06:06 GMT 2020
On Wed, 2020-09-30 at 09:43 +0200, Markus Böck wrote:
> Casting to intptr_t states the intent of an integer to pointer cast
> more clearly and ensures that the cast causes no loss of precision on
> any platforms. LLP64 platforms eg. have a long value of 4 bytes and
> pointer values of 8 bytes which may even cause compiler errors.
>
> Fixes PR 96608
>
> Would need this to be committed for me if accepted. (username
> zero9178, email markus.boeck02@gmail.com)
Sorry about the long delay on this; I got fixated on seeing if the
approach to how the analyzer hashes things needing changing, to make
the analyzer more deterministic, but I've now fixed the non-determinism
issue a different way.
The patch looks OK.
Successfully bootstrapped & regrtested on x86_64-pc-linux-gnu.
I've pushed it to master on your behalf as
942086bf73ee2ba6cfd7fdacc552940048437a6e.
Thanks
Dave
More information about the Gcc-patches
mailing list