Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Bugfix] Missing #include <sstream> #2629

Merged
merged 1 commit into from
Feb 20, 2019
Merged

[Bugfix] Missing #include <sstream> #2629

merged 1 commit into from
Feb 20, 2019

Conversation

hlu1
Copy link
Contributor

@hlu1 hlu1 commented Feb 20, 2019

std::ostringstream is only defined in <sstream> on some platforms.
@tqchen, please review

@Laurawly Laurawly self-assigned this Feb 20, 2019
@Laurawly Laurawly merged commit 9e53b3d into apache:master Feb 20, 2019
wweic pushed a commit to neo-ai/tvm that referenced this pull request Feb 20, 2019
@hlu1
[Bugfix] Missing #include <sstream> (apache#2629)
ea4587f 
Thanks @hlu1 and @tqchen, this is now merged.
@yzhliu yzhliu mentioned this pull request Mar 2, 2019
Closed
28 tasks
@hlu1 hlu1 deleted the sstream branch March 14, 2019 00:16
@yzhliu yzhliu mentioned this pull request Nov 11, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tqchen tqchen tqchen approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@hlu1 @tqchen @Laurawly