Skip to content

[SYCL] Return back 80 symbols limit for the SYCL LIT tests.#2149

Merged
bader merged 2 commits intointel:syclfrom
romanovvlad:private/vromanov/AdjustClangFormatConfigForTests
Jul 21, 2020
Merged

[SYCL] Return back 80 symbols limit for the SYCL LIT tests.#2149
bader merged 2 commits intointel:syclfrom
romanovvlad:private/vromanov/AdjustClangFormatConfigForTests

Conversation

@romanovvlad
Copy link
Contributor

But disable the limit for comments containing specific keywords:
RUN, FAIL, REQUIRES, UNSUPPORTED, CHECK, expected-*

But disable the limit for comments containing specific keywords:
RUN, FAIL, REQUIRES, UNSUPPORTED, CHECK, expected-*
@romanovvlad romanovvlad requested a review from bader July 21, 2020 09:23
@romanovvlad romanovvlad requested a review from a team as a code owner July 21, 2020 09:23
@romanovvlad romanovvlad requested a review from vladimirlaz July 21, 2020 09:23
@romanovvlad
Copy link
Contributor Author

We also can disable the limitation for all comments by setting ReflowComments: false

@romanovvlad
Copy link
Contributor Author

Dummy change in the run line passed the clang-format-check, which means the filter is working. I will revert this dummy change if the patch is approved.

vladimirlaz
vladimirlaz previously approved these changes Jul 21, 2020
@bader bader merged commit b7ae462 into intel:sycl Jul 21, 2020
Pennycook added a commit to Pennycook/llvm that referenced this pull request Jul 21, 2020
clang-format rules for tests changed in intel#2149

Signed-off-by: John Pennycook <john.pennycook@intel.com>
jsji pushed a commit that referenced this pull request Sep 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants