[PATCH] Less restrictive regex in const-uniq-1.c

Andrew Pinski pinskia@gmail.com
Tue Nov 6 16:39:00 GMT 2012


On Tue, Nov 6, 2012 at 8:07 AM, Andrew Pinski <pinskia@gmail.com> wrote:
> On Tue, Nov 6, 2012 at 6:54 AM, David Edelsohn <dje.gcc@gmail.com> wrote:
>> The regex in const-uniq-1.c assumes ELF label format, which does not
>> match AIX XCOFF.  The following patch broadens the regex so that it
>> also correctly matches on AIX.
>>
>>         * const-uniq-1.c: Expand regex to match AIX XCOFF labels.
>>
>> Index: const-uniq-1.c
>> ===================================================================
>> --- const-uniq-1.c      (revision 193203)
>> +++ const-uniq-1.c      (working copy)
>> @@ -20,5 +20,5 @@
>>    return a[i+1];
>>  }
>>
>> -/* { dg-final { scan-tree-dump-times "L\\\$?C0" 2 "gimple" } } */
>> +/* { dg-final { scan-tree-dump-times "L\\\$?C\\\.*0" 2 "gimple" } } */
>
> I think .* will match too much as it will match newlines too.  I think
> [^\n\]* is better.

Just to correct my self, I missed all of the \ before the '.' so this
change is fine with respect of not matching too much.

Thanks,
Andrew pinski


>
> Thanks,
> Andrew Pinski
>
>
>>  /* { dg-final { cleanup-tree-dump "gimple" } } */



More information about the Gcc-patches mailing list