diff options
author | Antonio Borneo <borneo.antonio@gmail.com> | 2020-11-15 22:10:58 +0100 |
---|---|---|
committer | Antonio Borneo <borneo.antonio@gmail.com> | 2020-12-05 23:18:37 +0000 |
commit | 1d3d87695c62be88d4a87c7d57de6084d654396b (patch) | |
tree | 9fd1f38c73a7ade3540d3226dc941612912f2d3a /src/target/register.h | |
parent | b5e015357ad4ae1fbb286f9bf6c22a563ab93eb7 (diff) | |
download | riscv-openocd-1d3d87695c62be88d4a87c7d57de6084d654396b.zip riscv-openocd-1d3d87695c62be88d4a87c7d57de6084d654396b.tar.gz riscv-openocd-1d3d87695c62be88d4a87c7d57de6084d654396b.tar.bz2 |
target/register: use an array of uint8_t for register's value
The use of 'void *' makes the pointer arithmetic incompatible with
standard C, even if this is allowed by GCC extensions.
The use of 'void *' can also hide incorrect pointer assignments.
Switch to 'uint8_t *' and add GCC warning flag to track any use of
pointer arithmetic extension.
Change-Id: Ic4d15a232834cd6b374330f70e2473a359b1607f
Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
Reviewed-on: http://openocd.zylin.com/5937
Tested-by: jenkins
Reviewed-by: Andreas Fritiofson <andreas.fritiofson@gmail.com>
Diffstat (limited to 'src/target/register.h')
-rw-r--r-- | src/target/register.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/target/register.h b/src/target/register.h index 1bae811..5f1c25f 100644 --- a/src/target/register.h +++ b/src/target/register.h @@ -127,7 +127,7 @@ struct reg { bool caller_save; /* Pointer to place where the value is stored, in the format understood by * the binarybuffer.h functions. */ - void *value; + uint8_t *value; /* The stored value needs to be written to the target. */ bool dirty; /* When true, value is valid. */ |