Skip to content

Update -sanitize=fuzzer option to take into account new libFuzzer location #11595

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
Aug 29, 2017

Conversation

cheshire
Copy link

This change will not pass tests yet, as it is dependent on llvm/clang/compiler-rt changes being cherry-picked (and those in turn will fail tests without this change).
Putting this out for code review.

@cheshire cheshire requested a review from devincoughlin August 24, 2017 02:05
@cheshire
Copy link
Author

Change is identical to #11594, but this one is on master, and other one is on master-next.

@gottesmm
Copy link
Contributor

@cheshire if you want to test the PRs together, you can use cross repo pull request testing. There is information on how to do this in ./docs/ContinuousIntegration.md.

@gottesmm
Copy link
Contributor

@cheshire
Copy link
Author

@gottesmm thanks, let's try that!

@cheshire
Copy link
Author

Please test with following PR:
apple/swift-llvm#57
apple/swift-clang#118
apple/swift-compiler-rt#12

@swift-ci Please smoke test

@cheshire
Copy link
Author

Please test with following PR:
apple/swift-llvm#57
apple/swift-clang#118
apple/swift-compiler-rt#12

@swift-ci Please smoke test

1 similar comment
@cheshire
Copy link
Author

Please test with following PR:
apple/swift-llvm#57
apple/swift-clang#118
apple/swift-compiler-rt#12

@swift-ci Please smoke test

@cheshire cheshire merged commit 020801b into swiftlang:master Aug 29, 2017
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