PR24028, PPC_INT_FMT

PPC_INT_FMT is redundant now that bfd.h pulls in inttypes.h if
available.  Apparently MacOS Mojave defines int64_t as long long even
though long is also 64 bits, which confuses the logic selecting
PPC_INT_FMT (and BFD_PRI64 too).  Hopefully inttypes.h is available on
Mojave.

	PR 24028
include/
	* opcode/ppc.h (PPC_INT_FMT): Delete.
opcodes/
	* ppc-dis.c (print_insn_powerpc): Replace PPC_INT_FMT uses with
	PRId64/PRIx64.

diff --git a/include/ChangeLog b/include/ChangeLog
index 5e6ed7e62b198de38ec768c9d989f0705822724d..53c423b5e9c78926a5733ba6e30e746ab31b4024 100644 (file)
--- a/include/ChangeLog
+++ b/include/ChangeLog
@@ -1,3 +1,8 @@
+2018-12-28  Alan Modra  <amodra@gmail.com>
+
+       PR 24028
+       * opcode/ppc.h (PPC_INT_FMT): Delete.
+
 2018-12-14  H.J. Lu  <hongjiu.lu@intel.com>
 
        PR ld/23900

diff --git a/include/opcode/ppc.h b/include/opcode/ppc.h
index c7262f1aefc680538a7c768739381f291e434ca3..c6b978eca3a1e3c4621fb099039a19e9cdaf3aaf 100644 (file)
--- a/include/opcode/ppc.h
+++ b/include/opcode/ppc.h
@@ -30,14 +30,6 @@ extern "C" {
 
 typedef uint64_t ppc_cpu_t;
 
-#if BFD_HOST_64BIT_LONG
-# define PPC_INT_FMT "l"
-#elif defined (__MSVCRT__)
-# define PPC_INT_FMT "I64"
-#else
-# define PPC_INT_FMT "ll"
-#endif
-
 /* The opcode table is an array of struct powerpc_opcode.  */
 
 struct powerpc_opcode

diff --git a/opcodes/ChangeLog b/opcodes/ChangeLog
index 8676f5f3b525a2fde3d898a848628f8a77db224f..4fa14fdb2680c1eea096bc3969e8ad8301d2983e 100644 (file)
--- a/opcodes/ChangeLog
+++ b/opcodes/ChangeLog
@@ -1,3 +1,9 @@
+2018-12-28  Alan Modra  <amodra@gmail.com>
+
+       PR 24028
+       * ppc-dis.c (print_insn_powerpc): Replace PPC_INT_FMT uses with
+       PRId64/PRIx64.
+
 2018-12-18  Alan Modra  <amodra@gmail.com>
 
        * arm-dis.c: Include bfd.h.

diff --git a/opcodes/ppc-dis.c b/opcodes/ppc-dis.c
index adbd12423fed901fd2f72d74e6477516baba731a..c618ecf518459be0c5568d1a88423b48cc59b66e 100644 (file)
--- a/opcodes/ppc-dis.c
+++ b/opcodes/ppc-dis.c
@@ -761,27 +761,27 @@ print_insn_powerpc (bfd_vma memaddr,
          /* Print the operand as directed by the flags.  */
          if ((operand->flags & PPC_OPERAND_GPR) != 0
              || ((operand->flags & PPC_OPERAND_GPR_0) != 0 && value != 0))
-           (*info->fprintf_func) (info->stream, "r%" PPC_INT_FMT "d", value);
+           (*info->fprintf_func) (info->stream, "r%" PRId64, value);
          else if ((operand->flags & PPC_OPERAND_FPR) != 0)
-           (*info->fprintf_func) (info->stream, "f%" PPC_INT_FMT "d", value);
+           (*info->fprintf_func) (info->stream, "f%" PRId64, value);
          else if ((operand->flags & PPC_OPERAND_VR) != 0)
-           (*info->fprintf_func) (info->stream, "v%" PPC_INT_FMT "d", value);
+           (*info->fprintf_func) (info->stream, "v%" PRId64, value);
          else if ((operand->flags & PPC_OPERAND_VSR) != 0)
-           (*info->fprintf_func) (info->stream, "vs%" PPC_INT_FMT "d", value);
+           (*info->fprintf_func) (info->stream, "vs%" PRId64, value);
          else if ((operand->flags & PPC_OPERAND_RELATIVE) != 0)
            (*info->print_address_func) (memaddr + value, info);
          else if ((operand->flags & PPC_OPERAND_ABSOLUTE) != 0)
            (*info->print_address_func) ((bfd_vma) value & 0xffffffff, info);
          else if ((operand->flags & PPC_OPERAND_FSL) != 0)
-           (*info->fprintf_func) (info->stream, "fsl%" PPC_INT_FMT "d", value);
+           (*info->fprintf_func) (info->stream, "fsl%" PRId64, value);
          else if ((operand->flags & PPC_OPERAND_FCR) != 0)
-           (*info->fprintf_func) (info->stream, "fcr%" PPC_INT_FMT "d", value);
+           (*info->fprintf_func) (info->stream, "fcr%" PRId64, value);
          else if ((operand->flags & PPC_OPERAND_UDI) != 0)
-           (*info->fprintf_func) (info->stream, "%" PPC_INT_FMT "d", value);
+           (*info->fprintf_func) (info->stream, "%" PRId64, value);
          else if ((operand->flags & PPC_OPERAND_CR_REG) != 0
                   && (((dialect & PPC_OPCODE_PPC) != 0)
                       || ((dialect & PPC_OPCODE_VLE) != 0)))
-           (*info->fprintf_func) (info->stream, "cr%" PPC_INT_FMT "d", value);
+           (*info->fprintf_func) (info->stream, "cr%" PRId64, value);
          else if (((operand->flags & PPC_OPERAND_CR_BIT) != 0)
                   && (((dialect & PPC_OPCODE_PPC) != 0)
                       || ((dialect & PPC_OPCODE_VLE) != 0)))
@@ -797,7 +797,7 @@ print_insn_powerpc (bfd_vma memaddr,
              (*info->fprintf_func) (info->stream, "%s", cbnames[cc]);
            }
          else
-           (*info->fprintf_func) (info->stream, "%" PPC_INT_FMT "d", value);
+           (*info->fprintf_func) (info->stream, "%" PRId64, value);
 
          if (need_paren)
            {
@@ -819,7 +819,7 @@ print_insn_powerpc (bfd_vma memaddr,
     }
 
   /* We could not find a match.  */
-  (*info->fprintf_func) (info->stream, ".long 0x%" PPC_INT_FMT "x", insn);
+  (*info->fprintf_func) (info->stream, ".long 0x%" PRIx64, insn);
 
   return 4;
 }