[PATCH, rs6000] Use $ instead of . for PC
Bill Schmidt
wschmidt@linux.vnet.ibm.com
Fri Jan 19 22:53:00 GMT 2018
> On Jan 19, 2018, at 3:58 PM, Segher Boessenkool <segher@kernel.crashing.org> wrote:
>
> On Fri, Jan 19, 2018 at 10:20:23PM +0100, Andreas Schwab wrote:
>> On Jan 19 2018, Bill Schmidt <wschmidt@linux.vnet.ibm.com> wrote:
>>
>>>> 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.
>>
>> Use braces.
>
> Yes, either
>
> +/* { dg-final { scan-assembler "b \\\$" } } */
>
> or (preferably)
>
> +/* { dg-final { scan-assembler {b \$} } } */
Thanks, going with the latter suggestion. My environment crapped out so I'm restarting
the regstrap. Will repost later this evening after it passes.
Thanks,
Bill
>
>
> Segher
More information about the Gcc-patches
mailing list