Re: [PATCH][AArch64] LR register not used in leaf functions

2014-11-15 Thread Jiong Wang
2014-11-15 15:49 GMT+00:00 Andrew Pinski : >> My local modifications can be found in the gcc git at >> apinski/thunderx-cost. Note I reverted this patch so I can continue >> working. The testcase is compiling newlib. Let me try to get it >> again. >> I was configuring a combined build with: >> -

Re: [PATCH][AArch64] LR register not used in leaf functions

2014-11-15 Thread Andrew Pinski
On Sat, Nov 15, 2014 at 7:21 AM, Andrew Pinski wrote: > On Sat, Nov 15, 2014 at 6:08 AM, Jiong Wang > wrote: >> 2014-11-15 0:15 GMT+00:00 Andrew Pinski : >>> On Tue, Sep 30, 2014 at 8:00 AM, Jiong Wang wrote: On 27/09/14 22:20, Kugan wrote: > > > On 23/09/14 01:58, Jiong Wang wr

Re: [PATCH][AArch64] LR register not used in leaf functions

2014-11-15 Thread Andrew Pinski
On Sat, Nov 15, 2014 at 6:08 AM, Jiong Wang wrote: > 2014-11-15 0:15 GMT+00:00 Andrew Pinski : >> On Tue, Sep 30, 2014 at 8:00 AM, Jiong Wang wrote: >>> On 27/09/14 22:20, Kugan wrote: On 23/09/14 01:58, Jiong Wang wrote: >> >> + /* If we decided that we didn't need a leaf fra

Re: [PATCH][AArch64] LR register not used in leaf functions

2014-11-15 Thread Jiong Wang
2014-11-15 0:15 GMT+00:00 Andrew Pinski : > On Tue, Sep 30, 2014 at 8:00 AM, Jiong Wang wrote: >> On 27/09/14 22:20, Kugan wrote: >>> >>> >>> On 23/09/14 01:58, Jiong Wang wrote: > > + /* If we decided that we didn't need a leaf frame pointer but then used > + LR in the function, then we'll w

Re: [PATCH][AArch64] LR register not used in leaf functions

2014-11-14 Thread Andrew Pinski
On Tue, Sep 30, 2014 at 8:00 AM, Jiong Wang wrote: > On 27/09/14 22:20, Kugan wrote: >> >> >> On 23/09/14 01:58, Jiong Wang wrote: >>> >>> On 22/09/14 16:43, Kugan wrote: >>> AArch64 has the same issue ARM had where the LR register was not used in leaf functions. This was reported in >>>

Re: [PATCH][AArch64] LR register not used in leaf functions

2014-11-11 Thread Marcus Shawcroft
On 30 September 2014 16:00, Jiong Wang wrote: > gcc/ > * config/aarch64/aarch64.h (CALL_USED_REGISTERS): Mark LR as caller-save. > (EPILOGUE_USES): Guard the check by epilogue_completed. > * config/aarch64/aarch64.c (aarch64_layout_frame): Explictly check for LR. > (aarch64_can_eliminate)

Re: [PATCH][AArch64] LR register not used in leaf functions

2014-11-04 Thread Jiong Wang
On 01/10/14 09:00, Kugan wrote: On 01/10/14 01:00, Jiong Wang wrote: On 27/09/14 22:20, Kugan wrote: On 23/09/14 01:58, Jiong Wang wrote: On 22/09/14 16:43, Kugan wrote: AArch64 has the same issue ARM had where the LR register was not used in leaf functions. This was reported in https://gcc

Re: [PATCH][AArch64] LR register not used in leaf functions

2014-10-01 Thread Kugan
On 01/10/14 01:00, Jiong Wang wrote: > On 27/09/14 22:20, Kugan wrote: >> >> On 23/09/14 01:58, Jiong Wang wrote: >>> On 22/09/14 16:43, Kugan wrote: >>> AArch64 has the same issue ARM had where the LR register was not used in leaf functions. This was reported in https://gcc.gn

Re: [PATCH][AArch64] LR register not used in leaf functions

2014-09-30 Thread Jiong Wang
On 27/09/14 22:20, Kugan wrote: On 23/09/14 01:58, Jiong Wang wrote: On 22/09/14 16:43, Kugan wrote: AArch64 has the same issue ARM had where the LR register was not used in leaf functions. This was reported in https://gcc.gnu.org/bugzilla/show_bug.cgi?id=42017. In AArch64, this test-case nee

Re: [PATCH][AArch64] LR register not used in leaf functions

2014-09-27 Thread Kugan
On 23/09/14 01:58, Jiong Wang wrote: > On 22/09/14 16:43, Kugan wrote: > >> AArch64 has the same issue ARM had where the LR register was not used in >> leaf functions. This was reported in >> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=42017. In AArch64, this >> test-case need to be added with

Re: [PATCH][AArch64] LR register not used in leaf functions

2014-09-22 Thread Jiong Wang
On 22/09/14 16:43, Kugan wrote: AArch64 has the same issue ARM had where the LR register was not used in leaf functions. This was reported in https://gcc.gnu.org/bugzilla/show_bug.cgi?id=42017. In AArch64, this test-case need to be added with more live ranges for the need for the LR_REGNUM. i.e

[PATCH][AArch64] LR register not used in leaf functions

2014-09-22 Thread Kugan
AArch64 has the same issue ARM had where the LR register was not used in leaf functions. This was reported in https://gcc.gnu.org/bugzilla/show_bug.cgi?id=42017. In AArch64, this test-case need to be added with more live ranges for the need for the LR_REGNUM. i.e test-case in the PR needs additiona