Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Siddhesh Poyarekar <siddhesh@redhat.com>
To: Jan Kratochvil <jan.kratochvil@redhat.com>
Cc: gdb-patches@sourceware.org
Subject: Re: [PATCH 0/4] bitpos expansion summary reloaded
Date: Sun, 21 Oct 2012 07:36:00 -0000	[thread overview]
Message-ID: <20121021130546.02ea680c@spoyarek> (raw)
In-Reply-To: <20121004071314.GA4292@host2.jankratochvil.net>

[-- Attachment #1: Type: text/plain, Size: 1208 bytes --]

Hi,

Here is a fix on top of the bitpos fixes based on the warnings
generated from gcc -Wconversion.  I have also attached the report for
review; I have not rebased since the last submission to ensure that
the line numbers don't go awry. Most of the extra warnings were either
unrelated or were the length parameter to
(store|extract)_(un)?signed_integer functions that are safe.

I have also verified that this does not cause any regressions in the
testsuite and that the gcc warnings generated after this were safe.

Regards,
Siddhesh

gdb/ChangeLog:

	* ada-lang.c (ada_lookup_struct_elt_type): Expand parameter
	DISPP to point to LONGEST.  Expand DISP to LONGEST.
	* breakpoint.c (insert_watchpoint): Expand LENGTH to LONGEST.
	(remove_watchpoint): Likewise.
	(resources_needed_watchpoint): Likewise.
	* hppa-tdep.c (hppa64_push_dummy_call): Expand OFFSET, REGNUM to
	LONGEST.
	* m32r-tdep.c (m32r_push_dummy_call): Expand STACK_ALLOC to
	LONGEST.
	* rs6000-aix-tdep.c (rs6000_push_dummy_call): Expand II to
	LONGEST.
	* tic6x-tdep.c (tic6x_push_dummy_call): Expand STACK_OFFSET to
	LONGEST.
	* valops.c (value_slice): Expand parameters LOWBOUND, LENGTH to
	LONGEST.
	* value.h (value_slice): Likewise.


[-- Attachment #2: extra-fixes.patch --]
[-- Type: text/x-patch, Size: 5423 bytes --]

diff --git a/gdb/ada-lang.c b/gdb/ada-lang.c
index 146e733..df499db 100644
--- a/gdb/ada-lang.c
+++ b/gdb/ada-lang.c
@@ -152,7 +152,7 @@ static struct symbol *find_old_style_renaming_symbol (const char *,
 						      struct block *);
 
 static struct type *ada_lookup_struct_elt_type (struct type *, char *,
-                                                int, int, int *);
+                                                int, int, LONGEST *);
 
 static struct value *evaluate_subexp_type (struct expression *, int *);
 
@@ -6766,7 +6766,7 @@ ada_value_struct_elt (struct value *arg, char *name, int no_err)
 
 static struct type *
 ada_lookup_struct_elt_type (struct type *type, char *name, int refok,
-                            int noerr, int *dispp)
+                            int noerr, LONGEST *dispp)
 {
   int i;
 
@@ -6811,7 +6811,7 @@ ada_lookup_struct_elt_type (struct type *type, char *name, int refok,
     {
       const char *t_field_name = TYPE_FIELD_NAME (type, i);
       struct type *t;
-      int disp;
+      LONGEST disp;
 
       if (t_field_name == NULL)
         continue;
diff --git a/gdb/breakpoint.c b/gdb/breakpoint.c
index a7d5e32..e5788eb 100644
--- a/gdb/breakpoint.c
+++ b/gdb/breakpoint.c
@@ -10309,7 +10309,7 @@ static int
 insert_watchpoint (struct bp_location *bl)
 {
   struct watchpoint *w = (struct watchpoint *) bl->owner;
-  int length = w->exact ? 1 : bl->length;
+  LONGEST length = w->exact ? 1 : bl->length;
 
   return target_insert_watchpoint (bl->address, length, bl->watchpoint_type,
 				   w->cond_exp);
@@ -10321,7 +10321,7 @@ static int
 remove_watchpoint (struct bp_location *bl)
 {
   struct watchpoint *w = (struct watchpoint *) bl->owner;
-  int length = w->exact ? 1 : bl->length;
+  LONGEST length = w->exact ? 1 : bl->length;
 
   return target_remove_watchpoint (bl->address, length, bl->watchpoint_type,
 				   w->cond_exp);
@@ -10363,7 +10363,7 @@ static int
 resources_needed_watchpoint (const struct bp_location *bl)
 {
   struct watchpoint *w = (struct watchpoint *) bl->owner;
-  int length = w->exact? 1 : bl->length;
+  LONGEST length = w->exact? 1 : bl->length;
 
   return target_region_ok_for_hw_watchpoint (bl->address, length);
 }
diff --git a/gdb/hppa-tdep.c b/gdb/hppa-tdep.c
index e66f17f..f8444bc 100644
--- a/gdb/hppa-tdep.c
+++ b/gdb/hppa-tdep.c
@@ -951,7 +951,8 @@ hppa64_push_dummy_call (struct gdbarch *gdbarch, struct value *function,
 {
   struct gdbarch_tdep *tdep = gdbarch_tdep (gdbarch);
   enum bfd_endian byte_order = gdbarch_byte_order (gdbarch);
-  int i, offset = 0;
+  int i;
+  LONGEST offset = 0;
   CORE_ADDR gp;
 
   /* "The outgoing parameter area [...] must be aligned at a 16-byte
@@ -965,7 +966,7 @@ hppa64_push_dummy_call (struct gdbarch *gdbarch, struct value *function,
       LONGEST len = TYPE_LENGTH (type);
       const bfd_byte *valbuf;
       bfd_byte fptrbuf[8];
-      int regnum;
+      LONGEST regnum;
 
       /* "Each parameter begins on a 64-bit (8-byte) boundary."  */
       offset = align_up (offset, 8);
diff --git a/gdb/m32r-tdep.c b/gdb/m32r-tdep.c
index 438f482..d424776 100644
--- a/gdb/m32r-tdep.c
+++ b/gdb/m32r-tdep.c
@@ -688,7 +688,8 @@ m32r_push_dummy_call (struct gdbarch *gdbarch, struct value *function,
 		      CORE_ADDR struct_addr)
 {
   enum bfd_endian byte_order = gdbarch_byte_order (gdbarch);
-  int stack_offset, stack_alloc;
+  int stack_offset;
+  LONGEST stack_alloc;
   int argreg = ARG1_REGNUM;
   int argnum;
   struct type *type;
diff --git a/gdb/rs6000-aix-tdep.c b/gdb/rs6000-aix-tdep.c
index 37353f6..6353aa8 100644
--- a/gdb/rs6000-aix-tdep.c
+++ b/gdb/rs6000-aix-tdep.c
@@ -196,7 +196,7 @@ rs6000_push_dummy_call (struct gdbarch *gdbarch, struct value *function,
 {
   struct gdbarch_tdep *tdep = gdbarch_tdep (gdbarch);
   enum bfd_endian byte_order = gdbarch_byte_order (gdbarch);
-  int ii;
+  LONGEST ii;
   LONGEST len = 0;
   int argno;			/* current argument number */
   LONGEST argbytes;		/* current argument byte */
diff --git a/gdb/tic6x-tdep.c b/gdb/tic6x-tdep.c
index 85fd433..915605a 100644
--- a/gdb/tic6x-tdep.c
+++ b/gdb/tic6x-tdep.c
@@ -895,7 +895,7 @@ tic6x_push_dummy_call (struct gdbarch *gdbarch, struct value *function,
 {
   int argreg = 0;
   int argnum;
-  int stack_offset = 4;
+  LONGEST stack_offset = 4;
   LONGEST references_offset = 4;
   CORE_ADDR func_addr = find_function_addr (function, NULL);
   enum bfd_endian byte_order = gdbarch_byte_order (gdbarch);
diff --git a/gdb/valops.c b/gdb/valops.c
index 94e8f67..89f3bb7 100644
--- a/gdb/valops.c
+++ b/gdb/valops.c
@@ -3737,7 +3737,7 @@ value_of_this_silent (const struct language_defn *lang)
    bound as the original ARRAY.  */
 
 struct value *
-value_slice (struct value *array, int lowbound, int length)
+value_slice (struct value *array, LONGEST lowbound, LONGEST length)
 {
   struct type *slice_range_type, *slice_type, *range_type;
   LONGEST lowerbound, upperbound;
diff --git a/gdb/value.h b/gdb/value.h
index 33199f3..2796262 100644
--- a/gdb/value.h
+++ b/gdb/value.h
@@ -936,7 +936,7 @@ extern void preserve_one_value (struct value *, struct objfile *, htab_t);
 
 extern struct value *varying_to_slice (struct value *);
 
-extern struct value *value_slice (struct value *, int, int);
+extern struct value *value_slice (struct value *, LONGEST, LONGEST);
 
 extern struct value *value_literal_complex (struct value *, struct value *,
 					    struct type *);

[-- Attachment #3: gcc-warnings.out.report --]
[-- Type: application/octet-stream, Size: 22254 bytes --]

SAFE: ada-lang.c:2239:34: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE: ada-lang.c:2484:39: warning: conversion to ‘gdb_byte’ from ‘unsigned int’ may alter its value [-Wconversion]
SAFE: ada-lang.c:3478:29: warning: conversion to ‘int’ from ‘size_t’ may alter its value [-Wconversion]
SAFE: ada-lang.c:4168:3: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: ada-lang.c:4496:26: warning: conversion to ‘int’ from ‘size_t’ may alter its value [-Wconversion]
FIXED(Expand disp, dispp): ada-lang.c:6822:20: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
FIXED: ada-lang.c:6834:24: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
FIXED: ada-lang.c:6864:28: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE: ada-lang.c:7416:3: warning: conversion to ‘short int’ from ‘int’ may alter its value [-Wconversion]
SAFE: ada-lang.c:7418:271: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: ada-lang.c:7651:11: warning: conversion to ‘short int’ from ‘int’ may alter its value [-Wconversion]
SAFE: ada-lang.c:7653:276: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: ada-lang.c:7696:3: warning: conversion to ‘short int’ from ‘int’ may alter its value [-Wconversion]
SAFE: ada-lang.c:7698:288: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: ada-lang.c:8712:45: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: alpha-tdep.c:540:48: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: amd64-tdep.c:2764:55: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: arm-tdep.c:3696:4: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE: arm-tdep.c:3701:11: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
UNRELATED: arm-tdep.c:3805:27: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
UNRELATED: arm-tdep.c:3808:3: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: arm-tdep.c:4258:45: warning: conversion to ‘short unsigned int’ from ‘long unsigned int’ may alter its value [-Wconversion]
UNRElATED: arm-tdep.c:4273:8: warning: conversion to ‘unsigned int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: arm-tdep.c:4277:42: warning: conversion to ‘short unsigned int’ from ‘long unsigned int’ may alter its value [-Wconversion]
UNRELATED: arm-tdep.c:4280:5: warning: conversion to ‘unsigned int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: arm-tdep.c:4319:42: warning: conversion to ‘short unsigned int’ from ‘long unsigned int’ may alter its value [-Wconversion]
UNRELATED: arm-tdep.c:4322:5: warning: conversion to ‘unsigned int’ from ‘long unsigned int’ may alter its value [-Wconversion]
UNRELATED: arm-tdep.c:5265:19: warning: conversion to ‘int’ from ‘CORE_ADDR’ may alter its value [-Wconversion]
UNRELATED: arm-tdep.c:5268:21: warning: conversion to ‘int’ from ‘CORE_ADDR’ may alter its value [-Wconversion]
UNRELATED: arm-tdep.c:5274:65: warning: conversion to ‘int’ from ‘CORE_ADDR’ may alter its value [-Wconversion]
UNRELATED: arm-tdep.c:5277:24: warning: conversion to ‘int’ from ‘CORE_ADDR’ may alter its value [-Wconversion]
UNRELATED: arm-tdep.c:5881:3: warning: conversion to ‘unsigned char:4’ from ‘unsigned int’ may alter its value [-Wconversion]
UNRELATED: arm-tdep.c:5882:3: warning: conversion to ‘unsigned char:1’ from ‘int’ may alter its value [-Wconversion]
UNRELATED: arm-tdep.c:5883:3: warning: conversion to ‘unsigned char:1’ from ‘int’ may alter its value [-Wconversion]
UNRELATED: arm-tdep.c:6037:3: warning: conversion to ‘unsigned char:4’ from ‘unsigned int’ may alter its value [-Wconversion]
UNRELATED: arm-tdep.c:6038:3: warning: conversion to ‘unsigned char:1’ from ‘int’ may alter its value [-Wconversion]
UNRELATED: arm-tdep.c:6538:3: warning: conversion to ‘unsigned char:1’ from ‘int’ may alter its value [-Wconversion]
UNRELATED: arm-tdep.c:6979:3: warning: conversion to ‘unsigned char:1’ from ‘int’ may alter its value [-Wconversion]
UNRELATED: arm-tdep.c:6980:3: warning: conversion to ‘unsigned char:1’ from ‘int’ may alter its value [-Wconversion]
UNRELATED: arm-tdep.c:6981:3: warning: conversion to ‘unsigned char:1’ from ‘int’ may alter its value [-Wconversion]
UNRELATED: arm-tdep.c:6982:41: warning: conversion to ‘unsigned char:4’ from ‘unsigned int’ may alter its value [-Wconversion]
UNRELATED: arm-tdep.c:7092:46: warning: conversion to ‘unsigned char:1’ from ‘short unsigned int’ may alter its value [-Wconversion]
UNRELATED: arm-tdep.c:7093:43: warning: conversion to ‘unsigned char:1’ from ‘short unsigned int’ may alter its value [-Wconversion]
UNRELATED: arm-tdep.c:7094:3: warning: conversion to ‘unsigned char:1’ from ‘int’ may alter its value [-Wconversion]
UNRELATED: arm-tdep.c:7930:53: warning: conversion to ‘unsigned int’ from ‘long int’ may alter its value [-Wconversion]
UNRELATED: arm-tdep.c:7933:35: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
UNRELATED: arm-tdep.c:8011:32: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
UNRELATED: arm-tdep.c:8015:28: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: arm-tdep.c:9163:13: warning: conversion to ‘int’ from ‘size_t’ may alter its value [-Wconversion]
SAFE: avr-tdep.c:1289:11: warning: conversion to ‘int’ from ‘ssize_t’ may alter its value [-Wconversion]
SAFE: avr-tdep.c:304:42: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: avr-tdep.c:310:42: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: avr-tdep.c:321:44: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
FIXED(Expand length): breakpoint.c:10312:29: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
FIXED: breakpoint.c:10324:29: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
FIXED: breakpoint.c:10366:28: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE(MEMBERPTR type): cp-valprint.c:795:11: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE(printable character type): c-valprint.c:191:12: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: dwarf2expr.c:1056:38: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: dwarf2expr.c:265:35: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: dwarf2loc.c:1525:8: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: eval.c:524:3: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE: eval.c:524:64: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE: eval.c:524:3: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE: eval.c:524:64: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE: findvar.c:333:47: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: findvar.c:342:45: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: findvar.c:353:38: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: findvar.c:362:36: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: findvar.c:465:59: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: frv-tdep.c:1123:19: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE(Structs passed as pointers): frv-tdep.c:1249:23: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: frv-tdep.c:1331:19: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: gnu-v3-abi.c:534:23: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: gnu-v3-abi.c:537:25: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: gnu-v3-abi.c:645:12: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: gnu-v3-abi.c:655:56: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
UNRElATED: gnu-v3-abi.c:758:35: warning: conversion to ‘hashval_t’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: h8300-tdep.c:697:12: warning: conversion to ‘int’ from ‘ssize_t’ may alter its value [-Wconversion]
SAFE: h8300-tdep.c:796:45: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: h8300-tdep.c:860:53: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: h8300-tdep.c:864:53: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: h8300-tdep.c:890:53: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: h8300-tdep.c:894:53: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: hppa-tdep.c:1052:43: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: hppa-tdep.c:1069:11: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
FIXED(Expand offset): hppa-tdep.c:1075:14: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE: i386-darwin-tdep.c:132:18: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: i386-nat.c:531:54: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE: ia64-tdep.c:3241:11: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: ia64-tdep.c:3258:45: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: ia64-tdep.c:3306:11: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: ia64-tdep.c:3750:31: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE: ia64-tdep.c:3823:9: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE: iq2000-tdep.c:118:38: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: iq2000-tdep.c:96:44: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: lm32-tdep.c:285:59: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: lm32-tdep.c:323:45: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: lm32-tdep.c:352:19: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: m32c-tdep.c:2515:38: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: m32c-tdep.c:2530:46: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
FIXED(Expand stack_alloc): m32r-tdep.c:719:17: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: m68hc11-tdep.c:1194:12: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: m88k-tdep.c:286:14: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE: mep-tdep.c:2149:18: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: mep-tdep.c:2177:22: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: mep-tdep.c:2345:65: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: mips-tdep.c:4361:10: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE: mips-tdep.c:4367:9: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE: mips-tdep.c:4446:8: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE: mips-tdep.c:5290:8: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE: mips-tdep.c:5757:10: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE: mips-tdep.c:5761:9: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE: mips-tdep.c:5779:4: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE: mips-tdep.c:7791:45: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: mt-tdep.c:374:43: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: ppc-sysv-tdep.c:1768:46: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: ppc-sysv-tdep.c:1831:16: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: ppc-sysv-tdep.c:1870:24: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: ppc-sysv-tdep.c:824:43: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: ppc-sysv-tdep.c:883:16: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: p-valprint.c:109:8: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: p-valprint.c:159:15: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: p-valprint.c:214:11: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE: p-valprint.c:312:61: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: p-valprint.c:324:12: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE: python/py-value.c:983:12: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: rl78-tdep.c:731:38: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: rl78-tdep.c:743:44: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: rs6000-aix-tdep.c:300:42: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
FIXED(Expand ii): rs6000-aix-tdep.c:368:7: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
FIXED: rs6000-aix-tdep.c:394:7: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE: rs6000-aix-tdep.c:503:46: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: s390-tdep.c:2539:44: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: s390-tdep.c:2542:42: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: score-tdep.c:588:25: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE: sh-tdep.c:1134:21: warning: conversion to ‘int’ from ‘ssize_t’ may alter its value [-Wconversion]
SAFE: sh-tdep.c:1140:8: warning: conversion to ‘int’ from ‘ssize_t’ may alter its value [-Wconversion]
SAFE: sh-tdep.c:1156:12: warning: conversion to ‘int’ from ‘ssize_t’ may alter its value [-Wconversion]
SAFE: sh-tdep.c:1164:8: warning: conversion to ‘int’ from ‘ssize_t’ may alter its value [-Wconversion]
SAFE: sh-tdep.c:1261:21: warning: conversion to ‘int’ from ‘ssize_t’ may alter its value [-Wconversion]
SAFE: sh-tdep.c:1267:8: warning: conversion to ‘int’ from ‘ssize_t’ may alter its value [-Wconversion]
SAFE: sh-tdep.c:935:17: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: sparc64-tdep.c:797:21: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE(Structure or union arguments are always <= 16): sparc64-tdep.c:974:15: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE: sparc-tdep.c:494:17: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE: sparc-tdep.c:504:17: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE: spu-tdep.c:1387:14: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE: spu-tdep.c:409:38: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: spu-tdep.c:420:44: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: stap-probe.c:1221:50: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: stap-probe.c:1230:40: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
UNRELATED: target.c:1515:5: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
UNRELATED: target.c:1517:26: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: tic6x-tdep.c:1024:11: warning: conversion to ‘int’ from ‘ssize_t’ may alter its value [-Wconversion]
FIXED(Expand stack_offset): tic6x-tdep.c:936:20: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE: tic6x-tdep.c:984:15: warning: conversion to ‘int’ from ‘ssize_t’ may alter its value [-Wconversion]
UNRELATED: tracepoint.c:3993:24: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
UNRELATED: tracepoint.c:3995:20: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
UNRELATED: tracepoint.c:4084:7: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
UNRELATED: tracepoint.c:4139:50: warning: conversion to ‘int’ from ‘size_t’ may alter its value [-Wconversion]
UNRELATED: tracepoint.c:4144:3: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
UNRELATED: utils.c:3615:13: warning: conversion to ‘int’ from ‘size_t’ may alter its value [-Wconversion]
SAFE: valarith.c:1062:21: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: valarith.c:1210:25: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: valarith.c:1340:23: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: valops.c:1014:56: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
UNRELATED: valops.c:3175:15: warning: conversion to ‘unsigned int’ from ‘size_t’ may alter its value [-Wconversion]
UNRELATED: valops.c:3177:11: warning: conversion to ‘unsigned int’ from ‘long int’ may alter its value [-Wconversion]
SAFE: valops.c:503:38: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: valprint.c:852:53: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
UNRELATED: value.c:1206:3: warning: conversion to ‘unsigned char:1’ from ‘int’ may alter its value [-Wconversion]
SAFE: value.c:2449:37: warning: conversion to ‘long int’ from ‘DOUBLEST’ may alter its value [-Wconversion]
SAFE: value.c:2454:34: warning: conversion to ‘long int’ from ‘DOUBLEST’ may alter its value [-Wconversion]
SAFE: value.c:3019:3: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE: value.c:3028:3: warning: conversion to ‘int’ from ‘long int’ may alter its value [-Wconversion]
SAFE: xstormy16-tdep.c:625:44: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: xstormy16-tdep.c:650:38: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: xtensa-tdep.c:1828:60: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: xtensa-tdep.c:1830:58: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]
SAFE: xtensa-tdep.c:1835:56: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion]

  reply	other threads:[~2012-10-21  7:36 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-27 13:33 Siddhesh Poyarekar
2012-09-28 11:20 ` Jan Kratochvil
2012-09-28 11:40   ` Siddhesh Poyarekar
2012-09-28 12:06     ` Jan Kratochvil
2012-09-28 12:19       ` Siddhesh Poyarekar
2012-09-29 17:39 ` Jan Kratochvil
2012-09-29 18:12   ` Jan Kratochvil
2012-09-30  6:52     ` Jan Kratochvil
2012-10-01  5:21       ` Siddhesh Poyarekar
2012-10-01  6:14         ` Jan Kratochvil
2012-10-03 13:12       ` Siddhesh Poyarekar
2012-10-03 18:38         ` Jan Kratochvil
2012-10-04  7:20           ` Siddhesh Poyarekar
2012-10-03 19:56         ` Jan Kratochvil
2012-10-04  7:13           ` Jan Kratochvil
2012-10-21  7:36             ` Siddhesh Poyarekar [this message]
2012-10-22 20:45               ` Tom Tromey
2012-10-23  1:34                 ` Jan Kratochvil
2012-10-23  1:58                   ` Jan Kratochvil
2012-10-23  2:29                     ` Siddhesh Poyarekar
2012-10-23  2:37                       ` Jan Kratochvil
2012-10-23  2:38                   ` Tom Tromey
2012-10-23 19:11               ` Jan Kratochvil
2012-10-24 18:33                 ` Tom Tromey
2012-10-24 18:55                   ` Jan Kratochvil
2012-10-24 20:18                     ` Tom Tromey
2012-10-25 15:54               ` Jan Kratochvil
2012-10-25 16:52                 ` Siddhesh Poyarekar
2012-11-06 20:01                   ` Jan Kratochvil
2012-11-07 13:48                     ` Jan Kratochvil
2012-11-13 19:46                       ` Tom Tromey
2012-11-13 19:55                         ` Jan Kratochvil
2012-11-01 15:24               ` Jan Kratochvil
2012-11-01 16:56                 ` Siddhesh Poyarekar

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20121021130546.02ea680c@spoyarek \
    --to=siddhesh@redhat.com \
    --cc=gdb-patches@sourceware.org \
    --cc=jan.kratochvil@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox