RE: [RFC: Patch, PR 60158] gcc/varasm.c : Pass actual alignment value to output_constant_pool_2

2015-03-27 Thread rohitarul...@freescale.com
gt; The subject line and thread may confuse people that this is a PowerPC- > specific issue. You need approval from a reviewer with authority over > varasm.c. > > Thanks, David > > On Thu, May 8, 2014 at 9:54 AM, rohitarul...@freescale.com > wrote: > >>

RE: [RFC: Patch, PR 60158] gcc/varasm.c : Pass actual alignment value to output_constant_pool_2

2015-04-28 Thread rohitarul...@freescale.com
> .data.rel.ro.local entries. > > Rohit, > > The subject line and thread may confuse people that this is a PowerPC- > specific issue. You need approval from a reviewer with authority over > varasm.c. > > Thanks, David > > On Thu, May 8, 2014 at 9:54 AM, roh

RE: [RFC: Patch, PR 60158] gcc/varasm.c : Pass actual alignment value to output_constant_pool_2

2015-04-28 Thread rohitarul...@freescale.com
Patch, PR 60158] gcc/varasm.c : Pass actual alignment value >to output_constant_pool_2 > >On 04/28/2015 03:44 AM, rohitarul...@freescale.com wrote: >> Ping. >> >> -Original Message- >> From: Dharmakan Rohit-B30502 >> Sent: Friday, March 27, 2015 7:57

RE: [RFC: Patch, PR 60158] gcc/varasm.c : Pass actual alignment value to output_constant_pool_2

2015-04-29 Thread rohitarul...@freescale.com
[RFC: Patch, PR 60158] gcc/varasm.c : Pass actual alignment value > to output_constant_pool_2 > > On 04/28/2015 12:38 PM, rohitarul...@freescale.com wrote: > > > > > >> -Original Message- > >> From: Jeff Law [mailto:l...@redhat.com] > >> Sent:

RE: [RFC: Patch, PR 60158] gcc/varasm.c : Pass actual alignment value to output_constant_pool_2

2015-04-30 Thread rohitarul...@freescale.com
[RFC: Patch, PR 60158] gcc/varasm.c : Pass actual alignment value > to output_constant_pool_2 > > On 04/29/2015 04:30 AM, rohitarul...@freescale.com wrote: > >> > > > > Jeff, I have made the changes as per your comments and attached the > patch. > > If the p

RE: [RFC: Patch, PR 60158] gcc/varasm.c : Pass actual alignment value to output_constant_pool_2

2015-05-05 Thread rohitarul...@freescale.com
> -Original Message- > From: Jeff Law [mailto:l...@redhat.com] > Sent: Thursday, April 30, 2015 9:14 PM > On 04/30/2015 09:34 AM, rohitarul...@freescale.com wrote: > > > > > >> -Original Message- > >> From: Jeff Law [mailto:l...@redhat.

RE: [RFC: Patch, PR 60102] [4.9/4.10 Regression] powerpc fp-bit ices@dwf_regno

2014-10-06 Thread rohitarul...@freescale.com
> From: Dharmakan Rohit-B30502 > Sent: Monday, September 29, 2014 3:54 PM > > > From: Ulrich Weigand [mailto:uweig...@de.ibm.com] Maciej W. Rozycki > > wrote: > > > On Mon, 4 Aug 2014, Edmar wrote: > > > > > > > Committed on trunk, revision 213596 Committed on 4.9 branch, > > > > revision 213597 >

RE: [RFC: Patch, PR 60102] [4.9/4.10 Regression] powerpc fp-bit ices@dwf_regno

2014-10-09 Thread rohitarul...@freescale.com
> -Original Message- > From: Maciej W. Rozycki [mailto:ma...@codesourcery.com] > To: Ulrich Weigand > Cc: Dharmakan Rohit-B30502; Wienskoski Edmar-RA8797; David Edelsohn; gcc- > patc...@gcc.gnu.org; Alan Modra; Jakub Jelinek > Subject: Re: [RFC: Patch, PR 60102] [4.9/4.10 Regression] power

[RFC: Patch, PR 60102] [4.9/4.10 Regression] powerpc fp-bit ices at dwf_regno

2014-07-07 Thread rohitarul...@freescale.com
Hello All, This is related to the following bug: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60102 I have tried to fix the e500v2 build on GCC v4.9.0 with the attached patch. Can you please review and comment on the changes especially DWARF_FRAME_REGNUM, DWARF_REG_TO_UNWIND_COLUMN definitions?

RE: [RFC: Patch, PR 60102] [4.9/4.10 Regression] powerpc fp-bit ices at dwf_regno

2014-07-22 Thread rohitarul...@freescale.com
Ping! > -Original Message- > From: Dharmakan Rohit-B30502 > Sent: Tuesday, July 08, 2014 8:13 AM > To: gcc-patches@gcc.gnu.org > Cc: Wienskoski Edmar-RA8797; dje@gmail.com; Alan Modra; Jakub Jelinek > Subject: [RFC: Patch, PR 60102] [4.9/4.10 Regression] powerpc fp-bit ices at > dwf_re

RE: [RFC: Patch, PR 60102] [4.9/4.10 Regression] powerpc fp-bit ices@dwf_regno

2014-07-31 Thread rohitarul...@freescale.com
Ulrich, Thanks for your comments, I have updated the patch accordingly. > > /* The SPE has an additional 32 synthetic registers, with DWARF debug > > info numbering for these registers starting at 1200. While > > eh_frame @@ -951,13 +952,14 @@ enum data_align { align_abi, align_opt, > >

RE: [RFC: Patch, PR 60102] [4.9/4.10 Regression] powerpc fp-bit ices@dwf_regno

2014-08-01 Thread rohitarul...@freescale.com
Hello Ulrich, Thanks. > > /* Use gcc hard register numbering for eh_frame. */ -#define > >DWARF_FRAME_REGNUM(REGNO) (REGNO) > >+#define DWARF_FRAME_REGNUM(REGNO) \ > >+ ((REGNO) >= FIRST_SPE_HIGH_REGNO ? ((REGNO) - > FIRST_SPE_HIGH_REGNO + > >+1200) : (REGNO)) > > Any reason for not using SPE_

RE: [RFC: Patch, PR 60102] [4.9/4.10 Regression] powerpc fp-bit ices@dwf_regno

2014-08-01 Thread rohitarul...@freescale.com
Jakub, > On Fri, Aug 01, 2014 at 06:03:56PM +0000, rohitarul...@freescale.com wrote: > > PR target/60102 > > --- libgcc/config/rs6000/linux-unwind.h (revision 213110) > +++ libgcc/config/rs6000/linux-unwind.h (working copy) > @@ -274,8 +274,8 @@ ppc_fallback_frame_

[Patch, PR 60158] Generate .fixup sections for .data.rel.ro.local entries.

2014-04-25 Thread rohitarul...@freescale.com
Hello All, This is related to the following bug: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60158 Test case: Refer below Target: e500v2 Command line options: -Os -fdata-sections -fpic -mrelocatable -mno-spe Notes: GCC v4.7.3 puts the address of "abc" into the GOT directly GCC v

RE: [Patch, PR 60158] Generate .fixup sections for .data.rel.ro.local entries.

2014-05-08 Thread rohitarul...@freescale.com
fixup sections for > .data.rel.ro.local entries. > > On Fri, Apr 25, 2014 at 02:57:38PM +, rohitarul...@freescale.com > wrote: > > Source file: gcc-4.8.2/gcc/varasm.c > > @@ -7120,7 +7120,7 @@ > >if (CONSTANT_POOL_ADDRESS_P (symbol)) > >

RE: [Patch, PR 60158] gcc/varasm.c : Pass actual alignment value to output_constant_pool_2

2014-06-04 Thread rohitarul...@freescale.com
over > varasm.c. > > Thanks, David > > On Thu, May 8, 2014 at 9:54 AM, rohitarul...@freescale.com > wrote: > >> -Original Message- > >> From: Alan Modra [mailto:amo...@gmail.com] > >> Sent: Saturday, April 26, 2014 11:52 AM > >>

RE: [RFC: Patch, PR 60102] [4.9/4.10 Regression] powerpc fp-bit ices@dwf_regno

2014-09-29 Thread rohitarul...@freescale.com
> From: Ulrich Weigand [mailto:uweig...@de.ibm.com] > Maciej W. Rozycki wrote: > > On Mon, 4 Aug 2014, Edmar wrote: > > > > > Committed on trunk, revision 213596 > > > Committed on 4.9 branch, revision 213597 > > > > This change regressed GDB for e500v2 multilibs, presumably because it > > does

RE: [RFC: Patch, PR 60102] [4.9/4.10 Regression] powerpc fp-bit ices@dwf_regno

2014-08-05 Thread rohitarul...@freescale.com
Jakub, > On Mon, Aug 04, 2014 at 11:51:34AM -0500, Edmar wrote: > > Committed on trunk, revision 213596 > > Committed on 4.9 branch, revision 213597 > > Note the ChangeLog entry was grossly misformated. > I've fixed it up in gcc/ChangeLog on the trunk, but not on the branch > nor in libgcc. The