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

Update link.x #61

Merged
merged 1 commit into from
Jul 15, 2020
Merged

Update link.x #61

merged 1 commit into from
Jul 15, 2020

Conversation

richardeoin
Copy link
Contributor

Similar to #38, riscv compilers may also generate 'small ro data' .srodata sections.

This doesn't appear to be well documented, but see this blog post for small amount of context.

@richardeoin richardeoin requested a review from a team as a code owner July 14, 2020 20:25
almindor
almindor previously approved these changes Jul 14, 2020
Copy link
Contributor

@almindor almindor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM @Disasm do you want to bump version in the PR too?

@Disasm
Copy link
Member

Disasm commented Jul 15, 2020

@richardeoin, looks good, thank you! Could you add a corresponding change log entry?

@almindor I think it's better to bump the version together with the release-relevant changelog modifications, so I'd rather do this in a separate PR.

Also we need yet another fix for this: rust-lang/rust#73564 (comment)

Similar to rust-embedded#38, riscv compilers may also generate 'small ro data' `.srodata` sections.

This doesn't appear to be well documented, but see [this blog post](https://www.sifive.com/blog/all-aboard-part-3-linker-relaxation-in-riscv-toolchain) for small amount of context.
@richardeoin
Copy link
Contributor Author

Done! Thanks for the eh_frame fix too, I hit that on recent nightlies

@richardeoin richardeoin requested a review from almindor July 15, 2020 20:56
Copy link
Member

@Disasm Disasm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@Disasm
Copy link
Member

Disasm commented Jul 15, 2020

bors r=almindor

@bors
Copy link
Contributor

bors bot commented Jul 15, 2020

Build succeeded:

@bors bors bot merged commit a1792e2 into rust-embedded:master Jul 15, 2020
@Disasm
Copy link
Member

Disasm commented Jul 16, 2020

Released in v0.7.2

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

Successfully merging this pull request may close these issues.

3 participants