This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, rs6000] Use $ instead of . for PC
- From: Bill Schmidt <wschmidt at linux dot vnet dot ibm dot com>
- To: Jakub Jelinek <jakub at redhat dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Segher Boessenkool <segher at kernel dot crashing dot org>, David Edelsohn <dje dot gcc at gmail dot com>
- Date: Fri, 19 Jan 2018 15:13:50 -0600
- Subject: Re: [PATCH, rs6000] Use $ instead of . for PC
- Authentication-results: sourceware.org; auth=none
- References: <43608c3f-c4ca-6fce-ea47-4cb2b6860ec5@linux.vnet.ibm.com> <20180119210900.GW2063@tucnak>
> On Jan 19, 2018, at 3:09 PM, Jakub Jelinek <jakub@redhat.com> wrote:
>
> On Fri, Jan 19, 2018 at 02:58:07PM -0600, Bill Schmidt wrote:
>> --- gcc/testsuite/gcc.target/powerpc/safe-indirect-jump-2.c (revision 256894)
>> +++ gcc/testsuite/gcc.target/powerpc/safe-indirect-jump-2.c (working copy)
>> @@ -30,4 +30,4 @@ int foo (int x)
>>
>> /* { dg-final { scan-assembler "crset 30" } } */
>> /* { dg-final { scan-assembler "beqctr- 7" } } */
>> -/* { dg-final { scan-assembler "b ." } } */
>> +/* { dg-final { scan-assembler "b $" } } */
>
> Does $ in scan-assembler really match a literal $ and not end of line?
> Looking around, most of scan-assembler patterns that want to match a $ use
> \\\$
Right. Working on getting the right number of backslashes in here...
I can never remember which ones need one and which need 3.
Thanks...
Bill
>
>> Index: gcc/testsuite/gcc.target/powerpc/safe-indirect-jump-3.c
>> ===================================================================
>> --- gcc/testsuite/gcc.target/powerpc/safe-indirect-jump-3.c (revision 256894)
>> +++ gcc/testsuite/gcc.target/powerpc/safe-indirect-jump-3.c (working copy)
>> @@ -49,4 +49,4 @@ int foo (int x)
>>
>> /* { dg-final { scan-assembler "crset 30" } } */
>> /* { dg-final { scan-assembler "beqctr- 7" } } */
>> -/* { dg-final { scan-assembler "b ." } } */
>> +/* { dg-final { scan-assembler "b $" } } */
>> Index: gcc/testsuite/gcc.target/powerpc/safe-indirect-jump-8.c
>> ===================================================================
>> --- gcc/testsuite/gcc.target/powerpc/safe-indirect-jump-8.c (revision 256894)
>> +++ gcc/testsuite/gcc.target/powerpc/safe-indirect-jump-8.c (working copy)
>> @@ -12,4 +12,4 @@ int bar ()
>>
>> /* { dg-final { scan-assembler "crset eq" } } */
>> /* { dg-final { scan-assembler "beqctr-" } } */
>> -/* { dg-final { scan-assembler "b ." } } */
>> +/* { dg-final { scan-assembler "b $" } } */
>> Index: gcc/testsuite/gcc.target/powerpc/safe-indirect-jump-2.c
>> ===================================================================
>> --- gcc/testsuite/gcc.target/powerpc/safe-indirect-jump-2.c (revision 256894)
>> +++ gcc/testsuite/gcc.target/powerpc/safe-indirect-jump-2.c (working copy)
>> @@ -30,4 +30,4 @@ int foo (int x)
>>
>> /* { dg-final { scan-assembler "crset 30" } } */
>> /* { dg-final { scan-assembler "beqctr- 7" } } */
>> -/* { dg-final { scan-assembler "b ." } } */
>> +/* { dg-final { scan-assembler "b $" } } */
>> Index: gcc/testsuite/gcc.target/powerpc/safe-indirect-jump-3.c
>> ===================================================================
>> --- gcc/testsuite/gcc.target/powerpc/safe-indirect-jump-3.c (revision 256894)
>> +++ gcc/testsuite/gcc.target/powerpc/safe-indirect-jump-3.c (working copy)
>> @@ -49,4 +49,4 @@ int foo (int x)
>>
>> /* { dg-final { scan-assembler "crset 30" } } */
>> /* { dg-final { scan-assembler "beqctr- 7" } } */
>> -/* { dg-final { scan-assembler "b ." } } */
>> +/* { dg-final { scan-assembler "b $" } } */
>> Index: gcc/testsuite/gcc.target/powerpc/safe-indirect-jump-8.c
>> ===================================================================
>> --- gcc/testsuite/gcc.target/powerpc/safe-indirect-jump-8.c (revision 256894)
>> +++ gcc/testsuite/gcc.target/powerpc/safe-indirect-jump-8.c (working copy)
>> @@ -12,4 +12,4 @@ int bar ()
>>
>> /* { dg-final { scan-assembler "crset eq" } } */
>> /* { dg-final { scan-assembler "beqctr-" } } */
>> -/* { dg-final { scan-assembler "b ." } } */
>> +/* { dg-final { scan-assembler "b $" } } */
>
> Jakub