Skip to content

master-next: update to match LLVM r320996 #13669

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
Merged
merged 1 commit into from
Jan 2, 2018
Merged

master-next: update to match LLVM r320996 #13669

merged 1 commit into from
Jan 2, 2018

Conversation

francisvm
Copy link
Contributor

The YAML mustQuote function now returns an enum instead of a bool.

rdar://problem/36022642

@bob-wilson @francisvm
master-next: update to match LLVM r320996
15e209e 
The YAML mustQuote function now returns an enum instead of a bool.
@francisvm francisvm requested a review from atrick January 2, 2018 11:21
@atrick
Copy link
Contributor

atrick commented Jan 2, 2018

@swift-ci test.

@dcci
Copy link
Member

dcci commented Jan 2, 2018

@swift-ci please test

@atrick
Copy link
Contributor

atrick commented Jan 2, 2018

I can't override the merge requirements. We just need to wait for tests.

@davezarzycki davezarzycki merged commit 65f1b82 into swiftlang:master Jan 2, 2018
@davezarzycki
Copy link
Contributor

I've merged this because:

  1. It is 6:30am PST and many Apple employees probably aren't awake yet.
  2. It is January 2nd and many Apple employees are probably still on vacation.
  3. @atrick's comments imply that he would have merged this ahead of the tests finishing anyway.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@atrick atrick Awaiting requested review from atrick

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

Successfully merging this pull request may close these issues.
5 participants
@francisvm @atrick @dcci @davezarzycki @bob-wilson