Skip to content

fix: align bytes ptr address for cacheable::from_byte#8450

Merged
jerrykingxyz merged 2 commits intomainfrom
jerry/miri
Nov 15, 2024
Merged

fix: align bytes ptr address for cacheable::from_byte#8450
jerrykingxyz merged 2 commits intomainfrom
jerry/miri

Conversation

@jerrykingxyz
Copy link
Copy Markdown
Contributor

Summary

  1. cacheable::from_byte align bytes ptr address. This will fix UnalignedPointer error in rkyv when miri test.
  2. skip cacheable_dyn miri test. The global register crate like inventory not work in miri test.

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@github-actions github-actions bot added the release: bug fix release: bug related release(mr only) label Nov 15, 2024
@github-actions github-actions bot added the team The issue/pr is created by the member of Rspack. label Nov 15, 2024
@netlify
Copy link
Copy Markdown

netlify bot commented Nov 15, 2024

Deploy Preview for rspack canceled.

Name Link
🔨 Latest commit 51e3f5b
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/673741b4cc456e0008d38214

h-a-n-a
h-a-n-a previously approved these changes Nov 15, 2024
@h-a-n-a h-a-n-a changed the title fix: cacheable::from_byte align bytes ptr address fix: cacheable::from_byte align bytes ptr address Nov 15, 2024
@h-a-n-a h-a-n-a changed the title fix: cacheable::from_byte align bytes ptr address fix: align bytes ptr address for cacheable::from_byte Nov 15, 2024
@jerrykingxyz jerrykingxyz merged commit 425deee into main Nov 15, 2024
@jerrykingxyz jerrykingxyz deleted the jerry/miri branch November 15, 2024 13:17
@h-a-n-a h-a-n-a mentioned this pull request Nov 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release: bug fix release: bug related release(mr only) team The issue/pr is created by the member of Rspack.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants