Skip to content
This repository was archived by the owner on Mar 28, 2020. It is now read-only.

Propose fix some typos #17

Closed
wants to merge 1 commit into from
Closed

Propose fix some typos #17

wants to merge 1 commit into from

Conversation

jeis2497052
Copy link

are these updates OK?

@jrose-apple
Copy link

Thanks! These fixes should go to the original compiler-rt, though – see http://compiler-rt.llvm.org.

@jeis2497052
Copy link
Author

OK, so an additional pull request to llvm.org original source of compiler-rt ?

@jrose-apple
Copy link

Yes please!

@jrose-apple
Copy link

The Swift project will then pick up your fixes at some point in the future, when we roll forward to a newer version of LLVM, Clang, and compiler-rt.

@jeis2497052
Copy link
Author

sounds good, looks like LLVM is stuck in subversion :-(

@delcypher
Copy link
Contributor

@jeis2497052 You can use git-svn. See http://llvm.org/docs/GettingStarted.html#for-developers-to-work-with-git-svn .

Please also take a look at http://llvm.org/docs/Phabricator.html . That document explains how how to use phabriactor which is what LLVM (and its sub-projects) uses for code review.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants