Skip to content

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

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

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