Re: PATCH: PR middle-end/49721: convert_memory_address_addr_space may generate invalid new insns

2011-08-26 Thread Richard Sandiford
H.J. Lu hjl.to...@gmail.com writes: On Sun, Aug 14, 2011 at 9:22 AM, H.J. Lu hjl.to...@gmail.com wrote: Hi, This patch is needed for x32 and only affects x32.  Any comments/objections to apply this to finish x32 support? Thanks. H.J. On Thu, Aug 11, 2011 at 6:25 AM, H.J. Lu

Re: PATCH: PR middle-end/49721: convert_memory_address_addr_space may generate invalid new insns

2011-08-19 Thread H.J. Lu
On Sun, Aug 14, 2011 at 9:22 AM, H.J. Lu hjl.to...@gmail.com wrote: Hi, This patch is needed for x32 and only affects x32.  Any comments/objections to apply this to finish x32 support? Thanks. H.J. On Thu, Aug 11, 2011 at 6:25 AM, H.J. Lu hjl.to...@gmail.com wrote: Hi, This is

Re: PATCH: PR middle-end/49721: convert_memory_address_addr_space may generate invalid new insns

2011-08-14 Thread H.J. Lu
Hi, This patch is needed for x32 and only affects x32. Any comments/objections to apply this to finish x32 support? Thanks. H.J. On Thu, Aug 11, 2011 at 6:25 AM, H.J. Lu hjl.to...@gmail.com wrote: Hi, This is the last patch needed for x32 support. convert_memory_address_addr_space

Re: PATCH: PR middle-end/49721: convert_memory_address_addr_space may generate invalid new insns

2011-08-11 Thread H.J. Lu
Hi, This is the last patch needed for x32 support. convert_memory_address_addr_space is called to convert a memory address without overflow/underflow. It should be safe to transform (zero_extend:DI (plus:SI (FOO:SI) (const_int Y))) to (plus:DI (zero_extend:DI (FOO:SI)) (const_int Y)) GCC

PATCH: PR middle-end/49721: convert_memory_address_addr_space may generate invalid new insns

2011-08-07 Thread H.J. Lu
Hi, We transform ptr_extend:DI (plus:SI (FOO:SI) (const_int Y))) to (plus:DI (ptr_extend:DI (FOO:SI)) (const_int Y)) since this is how Pmode != ptr_mode is supported even if the resulting address may overflow/underflow. It is also true for x32 which has zero_extend instead of ptr_extend. I

PATCH: PR middle-end/49721: convert_memory_address_addr_space may generate invalid new insns

2011-07-28 Thread H.J. Lu
On Thu, Jul 28, 2011 at 11:05 AM, Uros Bizjak ubiz...@gmail.com wrote: On Thu, Jul 28, 2011 at 7:59 PM, H.J. Lu hjl.to...@gmail.com wrote:  convert_memory_address_addr_space has a special PLUS/MULT case for  POINTERS_EXTEND_UNSIGNED  0. ?It turns out that it is also needed  for all Pmode

Re: PATCH: PR middle-end/49721: convert_memory_address_addr_space may generate invalid new insns

2011-07-28 Thread Uros Bizjak
On Thu, Jul 28, 2011 at 8:09 PM, H.J. Lu hjl.to...@gmail.com wrote:  convert_memory_address_addr_space has a special PLUS/MULT case for  POINTERS_EXTEND_UNSIGNED  0. ?It turns out that it is also needed  for all Pmode != ptr_mode cases. ?OK for trunk?  2011-06-11 ?H.J. Lu

Re: PATCH: PR middle-end/49721: convert_memory_address_addr_space may generate invalid new insns

2011-07-28 Thread H.J. Lu
On Thu, Jul 28, 2011 at 11:23 AM, Uros Bizjak ubiz...@gmail.com wrote: On Thu, Jul 28, 2011 at 8:09 PM, H.J. Lu hjl.to...@gmail.com wrote:  convert_memory_address_addr_space has a special PLUS/MULT case for  POINTERS_EXTEND_UNSIGNED  0. ?It turns out that it is also needed  for all Pmode

Re: PATCH: PR middle-end/49721: convert_memory_address_addr_space may generate invalid new insns

2011-07-28 Thread Uros Bizjak
On Thu, Jul 28, 2011 at 8:32 PM, H.J. Lu hjl.to...@gmail.com wrote:  convert_memory_address_addr_space has a special PLUS/MULT case for  POINTERS_EXTEND_UNSIGNED  0. ?It turns out that it is also needed  for all Pmode != ptr_mode cases. ?OK for trunk?  2011-06-11 ?H.J. Lu

Re: PATCH: PR middle-end/49721: convert_memory_address_addr_space may generate invalid new insns

2011-07-28 Thread H.J. Lu
On Thu, Jul 28, 2011 at 11:49 AM, Uros Bizjak ubiz...@gmail.com wrote: On Thu, Jul 28, 2011 at 8:32 PM, H.J. Lu hjl.to...@gmail.com wrote:  convert_memory_address_addr_space has a special PLUS/MULT case for  POINTERS_EXTEND_UNSIGNED  0. ?It turns out that it is also needed  for all Pmode