Re: [PATCH] [AArch64] Fix PR71112

2017-07-06 Thread Hurugalawadi, Naveen
Hi Ramana, >> PR71112 is still open - should this be backported to GCC-6 ? Ported the patch to gcc-6-branch and committed as:- https://gcc.gnu.org/viewcvs/gcc?view=revision=250014 Bootstrapped and Regression Tested gcc-6-branch for AArch64 on aarch64-thunder-linux. Thanks, Naveen

Re: [PATCH] [AArch64] Fix PR71112

2017-07-04 Thread Ramana Radhakrishnan
On Wed, Nov 23, 2016 at 5:25 AM, Hurugalawadi, Naveen wrote: > Hi, > > Please find attached the patch that fixes PR71112. > > The current implementation that handles SYMBOL_SMALL_GOT_28K in > aarch64_load_symref_appropriately access the high part of RTX for

Re: [PATCH] [AArch64] Fix PR71112

2016-12-07 Thread James Greenhalgh
On Wed, Dec 07, 2016 at 07:25:21AM +, Hurugalawadi, Naveen wrote: > Hi James, > > Thanks for the review and suggestions regarding the testcase. > > >> Why limit the ABI and endianness here, and if you do plan to do that > > Extra options have been dropped and the testcase will check across

Re: [PATCH] [AArch64] Fix PR71112

2016-12-06 Thread Hurugalawadi, Naveen
Hi James, Thanks for the review and suggestions regarding the testcase. >> Why limit the ABI and endianness here, and if you do plan to do that Extra options have been dropped and the testcase will check across all variants and endianness. Please find attached the modified patch as per the

Re: [PATCH] [AArch64] Fix PR71112

2016-12-06 Thread James Greenhalgh
On Wed, Nov 23, 2016 at 05:25:44AM +, Hurugalawadi, Naveen wrote: > Hi, > > Please find attached the patch that fixes PR71112. > > The current implementation that handles SYMBOL_SMALL_GOT_28K in > aarch64_load_symref_appropriately access the high part of RTX for Big-Endian > mode which

[PING][PATCH] [AArch64] Fix PR71112

2016-12-05 Thread Hurugalawadi, Naveen
Hi, Please consider this as a personal reminder to review the patch at following link and let me know your comments on the same. https://gcc.gnu.org/ml/gcc-patches/2016-11/msg02305.html Thanks, Naveen

Re: [PATCH] [AArch64] Fix PR71112

2016-12-05 Thread Kyrill Tkachov
[CC'ing James] On 23/11/16 05:25, Hurugalawadi, Naveen wrote: Hi, Please find attached the patch that fixes PR71112. The current implementation that handles SYMBOL_SMALL_GOT_28K in aarch64_load_symref_appropriately access the high part of RTX for Big-Endian mode which results in ICE for

[PATCH] [AArch64] Fix PR71112

2016-11-22 Thread Hurugalawadi, Naveen
Hi, Please find attached the patch that fixes PR71112. The current implementation that handles SYMBOL_SMALL_GOT_28K in aarch64_load_symref_appropriately access the high part of RTX for Big-Endian mode which results in ICE for ILP32. The attached patch modifies it by accessing the lower part for