Skip to content

Conversation

@v-klochkov
Copy link
Contributor

…usage

Signed-off-by: Vyacheslav N Klochkov [email protected]

@v-klochkov
Copy link
Contributor Author

Guys, please take a look at this fix. It is a 1 line fix that unblocks the build on MSVC-2017.
Using std::isspace requires "#include ", which I added in this fix.

Copy link
Contributor

@kbobrovs kbobrovs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The spec says, isspace is part of <cctype>, so LGTM

@vladimirlaz vladimirlaz merged commit 80c4b38 into intel:sycl Oct 24, 2019
@bader
Copy link
Contributor

bader commented Oct 24, 2019

@v-klochkov
Copy link
Contributor Author

@bader, thank you for the note. I have just created KhronosGroup/SPIRV-LLVM-Translator#364 there.

@v-klochkov v-klochkov deleted the public_vklochkov_cctype branch October 25, 2019 17:31
vladimirlaz pushed a commit that referenced this pull request Sep 29, 2020
Removed executable bit from test files

Moved some tests into separate directories to group them
by the extension they supposed to test.
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.

4 participants