Skip to content

[SPIRV] Include <cctype> to fix error on Windows due to std::isspace … #747

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
merged 1 commit into from
Oct 24, 2019

Conversation

v-klochkov
Copy link
Contributor

…usage

Signed-off-by: Vyacheslav N Klochkov vyacheslav.n.klochkov@intel.com

…usage

Signed-off-by: Vyacheslav N Klochkov <vyacheslav.n.klochkov@intel.com>
@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, is this patch committed to https://github.com/KhronosGroup/SPIRV-LLVM-Translator?

@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