Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@jrvanwhy
Copy link
Contributor

@jrvanwhy jrvanwhy commented Jul 12, 2021

Rendered

These meeting notes are from 9 days ago, not last Friday (sorry for the delay).

hudson-ayers
hudson-ayers previously approved these changes Jul 12, 2021
Comment on lines +16 to +25
* 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.
Copy link
Contributor

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.

@jrvanwhy
Copy link
Contributor Author

bors r+

@bors
Copy link
Contributor

bors bot commented Jul 18, 2021

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"}

@lschuermann
Copy link
Member

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+

@bors
Copy link
Contributor

bors bot commented Jul 19, 2021

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"}

@lschuermann
Copy link
Member

I just confirmed these builds run fine for me locally including make alldoc, so this is (as indicated in Netlify's) logs most likely unrelated to these changes.

@hudson-ayers @bradjc @ppannuto I can't merge manually. Can you merge this?

@hudson-ayers hudson-ayers merged commit d46f1ed into tock:master Jul 20, 2021
@jrvanwhy jrvanwhy deleted the core-notes-2021-07-02 branch July 14, 2022 21:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants