-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Update LLVM and update APIs #6147
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
Conversation
b5cfa3d to
7659a9a
Compare
9938ca3 to
53a2e98
Compare
chandlerc
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should now be ready to review. Sorry for confusion with earlier versions.
| http_archive( | ||
| name = "llvm-raw", | ||
| build_file_content = "# empty", | ||
| integrity = "sha256-6URudXFgqoCuQI4hbb7AlN4Wj7WzLHFB4AWGjUV0UAI=", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why remove this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because I was supposed to update it and I forgot to add that update back. Thanks for noticing, fixed.
This also updates the patch file for compiler-rt as upstream has changed a bit. No functional change.
fbd5a7e to
0b8bbef
Compare
This also updates the patch file for compiler-rt as upstream has changed a bit. No functional change.