r197542 - clang-format-diff.py: add the OpenCL file extension

Daniel Jasper djasper at google.com
Tue Dec 17 22:55:56 PST 2013


I think we should revert this. cl is also the default extension for common
lisp programs and we most definitely don't format those correctly (or
within reasonable amount of time).


On Wed, Dec 18, 2013 at 1:58 AM, Alp Toker <alp at nuanti.com> wrote:

> Author: alp
> Date: Tue Dec 17 18:58:58 2013
> New Revision: 197542
>
> URL: http://llvm.org/viewvc/llvm-project?rev=197542&view=rev
> Log:
> clang-format-diff.py: add the OpenCL file extension
>
> It's handled correctly as a C-family language.
>
> Modified:
>     cfe/trunk/tools/clang-format/clang-format-diff.py
>
> Modified: cfe/trunk/tools/clang-format/clang-format-diff.py
> URL:
> http://llvm.org/viewvc/llvm-project/cfe/trunk/tools/clang-format/clang-format-diff.py?rev=197542&r1=197541&r2=197542&view=diff
>
> ==============================================================================
> --- cfe/trunk/tools/clang-format/clang-format-diff.py (original)
> +++ cfe/trunk/tools/clang-format/clang-format-diff.py Tue Dec 17 18:58:58
> 2013
> @@ -47,7 +47,7 @@ def main():
>                        help='custom pattern selecting file paths to
> reformat '
>                        '(case sensitive, override -iregex)')
>    parser.add_argument('-iregex', metavar='PATTERN', default=
> -                      r'.*\.(cpp|cc|c\+\+|cxx|c|h|hpp|m|mm|inc|js)',
> +                      r'.*\.(cpp|cc|c\+\+|cxx|c|cl|h|hpp|m|mm|inc|js)',
>                        help='custom pattern selecting file paths to
> reformat '
>                        '(case insensitive, override -regex)')
>    parser.add_argument(
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20131218/72d8837c/attachment.html>


More information about the cfe-commits mailing list