Skip to content

[ClangImporter] Fix another lifetime extension issue. #5255

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

jrose-apple
Copy link
Contributor

The StringRef here referred to a temporary std::string.

rdar://problem/28699869

@jrose-apple
Copy link
Contributor Author

@swift-ci Please smoke test and merge

The StringRef here referred to a temporary std::string.

rdar://problem/28699869
@jrose-apple jrose-apple force-pushed the ClangImporter-StringRef-issue branch from 1a91f62 to 036520f Compare October 12, 2016 17:54
@jrose-apple
Copy link
Contributor Author

@swift-ci Please smoke test and merge

@lplarson
Copy link
Contributor

@jrose-apple Do you know what's going on with Linux here?

@jrose-apple
Copy link
Contributor Author

Hm, no, that doesn't look related. @ddunbar, does this look familiar?

@jrose-apple
Copy link
Contributor Author

@swift-ci Please smoke test

@jrose-apple
Copy link
Contributor Author

@shahmishal This one flagged the wrong commit again. https://ci.swift.org/job/swift-PR-Linux-smoke-test/1783/

@jrose-apple jrose-apple merged commit 4960ad1 into swiftlang:master Oct 13, 2016
@jrose-apple jrose-apple deleted the ClangImporter-StringRef-issue branch October 13, 2016 20:47
benlangmuir pushed a commit to benlangmuir/swift that referenced this pull request Oct 19, 2016
The StringRef here referred to a temporary std::string.

rdar://problem/28699869
Conflicts:
	lib/ClangImporter/ClangImporter.cpp
MaxDesiatov pushed a commit that referenced this pull request Sep 7, 2023
[pull] swiftwasm from main
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.

2 participants