On July 9, 2015 6:48:39 PM GMT+02:00, Jeff Law <l...@redhat.com> wrote:
>On 07/09/2015 10:30 AM, Bernhard Reutner-Fischer wrote:
>>
>> Thanks,
>>
>> gcc/ChangeLog
>>
>> 2015-07-09  Bernhard Reutner-Fischer  <al...@gcc.gnu.org>
>>
>>      * graphite.h: New file.
>>      (print_graphite_statistics): Extern declaration.
>>      * graphite-scop-detection.c (print_graphite_scop_statistics,
>>      print_graphite_statistics): Remove.
>>      * graphite.c (print_graphite_statistics): Make public and add
>>      PREFIX parameter. Adjust callers.
>>      (print_graphite_scop_statistics): Add PREFIX parameter.
>>
>> Signed-off-by: Bernhard Reutner-Fischer <rep.dot....@gmail.com>
>> ---
>> -/* Print statistics for SCOPS to FILE.  */
>> +/* Print statistics for SCOPS to FILE. Prefix statistic headers with
>PREFIX.  */
>Nit.  Two spaces after a period, even in comments.

Done.

I wasn't sure where to put the prototype, maybe a graphite.h for just that 
(ATM) is not warranted?
Maybe an extern decl in graphite-scop-detection.c would be sufficient?
>
>
>With that nit fixed, OK for the trunk after a bootstrap and regression
>test.

Thanks,
>
>jeff


Reply via email to