This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [COMMITTED][AArch64] Improve TLS Descriptor pattern to release RTL loop IV opt
- From: Jiong Wang <jiong dot wang at arm dot com>
- To: Andreas Schwab <schwab at linux-m68k dot org>
- Cc: James Greenhalgh <James dot Greenhalgh at arm dot com>, gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 10 Aug 2015 10:34:38 +0100
- Subject: Re: [COMMITTED][AArch64] Improve TLS Descriptor pattern to release RTL loop IV opt
- Authentication-results: sourceware.org; auth=none
- References: <n99bnewfm4l dot fsf at arm dot com> <20150804094851 dot GA12175 at arm dot com> <n99mvy4s8d9 dot fsf at arm dot com> <87io8q6le7 dot fsf at igel dot home>
Andreas Schwab writes:
> Jiong Wang <jiong.wang@arm.com> writes:
>
>> Index: gcc/ChangeLog
>> ===================================================================
>> --- gcc/ChangeLog (revision 226682)
>> +++ gcc/ChangeLog (working copy)
>> @@ -1,3 +1,16 @@
>> +2015-08-06 Ramana Radhakrishnan <ramana.radhakrishnan@arm.com>
>> + Jiong Wang <jiong.wang@arm.com>
>> +
>> + * config/aarch64/aarch64.d (tlsdesc_small_pseudo_<mode>): New pattern.
>> + * config/aarch64/aarch64.h (reg_class): New enumeration FIXED_REG0.
>> + (REG_CLASS_NAMES): Likewise.
>> + (REG_CLASS_CONTENTS): Likewise.
>> + * config/aarch64/aarch64.c (aarch64_class_max_nregs): Likewise.
>> + (aarch64_register_move_cost): Likewise.
>> + (aarch64_load_symref_appropriately): Invoke the new added pattern if
>> + possible.
>> + * config/aarch64/constraints.md (Uc0): New constraint.
>
> That breaks go, all tests are crashing now.
Andreas,
Thanks for the information.
* I found I committed the wrong patch!
there are two patches in my local directory, one is
"tlsdesc_hoist.patch" the other is "tlsdesc-hoist.patch", the one
approved and up-to-date is tlsdesc-hoist.patch while I committed
tlsdesc_hoist.patch.
Reverted the wrong commit and committed the correct/approved
version.
* Even after the correct patch applied, I still found go check failed
on my local native check.
Tring to understand why and if I can't figure out today I will
revert the patch.
Sorry about the trouble!
--
Regards,
Jiong