[RFC PATCH 1/5] RISC-V: Use bool on riscv_set_options members
Tsukasa OI
research_trasio@irq.a4lg.com
Thu Aug 11 07:00:49 GMT 2022
All uses of riscv_set_options members are safe to be bool. If we change
them from `int' to `bool', we can shrink the `riscv_set_options' and the
intent of the members are clearer.
gas/ChangeLog:
* config/tc-riscv.c (struct riscv_set_options): Make all members
from `int' to `bool'.
---
gas/config/tc-riscv.c | 20 ++++++++++----------
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/gas/config/tc-riscv.c b/gas/config/tc-riscv.c
index 34ce68e8252..92eabf0cbc7 100644
--- a/gas/config/tc-riscv.c
+++ b/gas/config/tc-riscv.c
@@ -229,20 +229,20 @@ riscv_set_default_priv_spec (const char *s)
/* This is the set of options which the .option pseudo-op may modify. */
struct riscv_set_options
{
- int pic; /* Generate position-independent code. */
- int rvc; /* Generate RVC code. */
- int relax; /* Emit relocs the linker is allowed to relax. */
- int arch_attr; /* Emit architecture and privileged elf attributes. */
- int csr_check; /* Enable the CSR checking. */
+ bool pic; /* Generate position-independent code. */
+ bool rvc; /* Generate RVC code. */
+ bool relax; /* Emit relocs the linker is allowed to relax. */
+ bool arch_attr; /* Emit architecture and privileged elf attributes. */
+ bool csr_check; /* Enable the CSR checking. */
};
static struct riscv_set_options riscv_opts =
{
- 0, /* pic */
- 0, /* rvc */
- 1, /* relax */
- DEFAULT_RISCV_ATTR, /* arch_attr */
- 0, /* csr_check */
+ false, /* pic */
+ false, /* rvc */
+ true, /* relax */
+ (bool) DEFAULT_RISCV_ATTR, /* arch_attr */
+ false, /* csr_check */
};
/* Enable or disable the rvc flags for riscv_opts. Turn on the rvc flag
--
2.34.1
More information about the Binutils
mailing list