Hi all,

I tried building a cc1 for riscv-x-elf and I got a build error about INT16_MAX 
not being defined on my system.
I think it's not a standard macro.  This patch fixes that with what I think is 
the equivalent definition using
GCC modes logic.

With this my build proceeds to build a cc1 successfully. No testing beyond that.

Is this ok for trunk?

Thanks,
Kyrill

P.S. Palmer, Andrew, I didn't see your names and emails in the MAINTAINERS 
file, you should update that when
you get the chance.


2016-02-07  Kyrylo Tkachov  <kyrylo.tkac...@arm.com>

    * config/riscv/riscv.c (riscv_build_integer_1): Avoid use of INT16_MAX.
diff --git a/gcc/config/riscv/riscv.c b/gcc/config/riscv/riscv.c
index 834651f..89567f7 100644
--- a/gcc/config/riscv/riscv.c
+++ b/gcc/config/riscv/riscv.c
@@ -356,7 +356,9 @@ riscv_build_integer_1 (struct riscv_integer_op codes[RISCV_MAX_INTEGER_OPS],
   /* End with ADDI.  When constructing HImode constants, do not generate any
      intermediate value that is not itself a valid HImode constant.  The
      XORI case below will handle those remaining HImode constants.  */
-  if (low_part != 0 && (mode != HImode || value - low_part <= INT16_MAX))
+  if (low_part != 0
+      && (mode != HImode
+	  || value - low_part <= ((1 << (GET_MODE_BITSIZE (HImode) - 1)) - 1)))
     {
       alt_cost = 1 + riscv_build_integer_1 (alt_codes, value - low_part, mode);
       if (alt_cost < cost)

Reply via email to