-
-
Notifications
You must be signed in to change notification settings - Fork 779
Add core WG notes from 2021-07-02 #2664
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
c129dff to
9386f02
Compare
| * Johnathan: Unfortunately this only applies to RISC-V, at least with the | ||
| current plans. On behalf of OpenTitan, LowRISC is working on an embedded | ||
| position-independent-code spec for RISC-V that they're contributing to LLVM | ||
| for both `clang` and `rustc`. | ||
| * Amit: That's awesome. Do we know if it's going to support the kind of | ||
| PIC we rely on right now in ARM for C? | ||
| * Johnathan: The current plans are such that it will initially look a lot like | ||
| that, but then it will change to be somewhat more efficient and eliminate the | ||
| global offset table. It was specifically designed for Tock's usage. | ||
| * Amit: That would be huge, that's great. |
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 is incredibly exciting. All the more so if it's being designed for OpenTitan explicitly. My memory from when we were first exploring this way back when is that ARM PIC does a bunch of little inefficient things that are unnecessary from a Tock perspective. The GOT being the biggest example.
|
bors r+ |
|
This PR was included in a batch that successfully built, but then failed to merge into master. It will not be retried. Additional information: {"message":"Required status check \"netlify/docs-tockosorg/deploy-preview\" is failing.","documentation_url":"https://docs.github.com/articles/about-protected-branches"} |
|
Netlify seems to have its share of issues in the past few days, trying again given https://www.netlifystatus.com/ says it's a go. bors r+ |
|
This PR was included in a batch that successfully built, but then failed to merge into master. It will not be retried. Additional information: {"message":"Required status check \"netlify/docs-tockosorg/deploy-preview\" is failing.","documentation_url":"https://docs.github.com/articles/about-protected-branches"} |
|
I just confirmed these builds run fine for me locally including @hudson-ayers @bradjc @ppannuto I can't merge manually. Can you merge this? |
Rendered
These meeting notes are from 9 days ago, not last Friday (sorry for the delay).