Skip to content

fix: handle potentially unaligned frame pointer#217

Merged
YangKeao merged 1 commit intotikv:masterfrom
gen-xu:gen/fix_misaligned_dereference
Jul 25, 2023
Merged

fix: handle potentially unaligned frame pointer#217
YangKeao merged 1 commit intotikv:masterfrom
gen-xu:gen/fix_misaligned_dereference

Conversation

@gen-xu
Copy link
Copy Markdown
Contributor

@gen-xu gen-xu commented Jul 14, 2023

this PR addressed the issue #216

@gen-xu gen-xu force-pushed the gen/fix_misaligned_dereference branch 2 times, most recently from 9ae5a39 to d715918 Compare July 14, 2023 07:24
@alindima
Copy link
Copy Markdown

alindima commented Jul 25, 2023

@YangKeao this is an important PR that prevents using pprof-rs with frame pointer unwinding, could you please approve it?

Copy link
Copy Markdown
Member

@YangKeao YangKeao left a comment

Choose a reason for hiding this comment

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

LGTM

@YangKeao
Copy link
Copy Markdown
Member

YangKeao commented Jul 25, 2023

Sorry for the late reply and thanks for your contribution! Once you signed-off the commit, I'll merge this pull request ❤️ .

You could sign-off through git commit --amend -s

Signed-off-by: Gen Xu <xgbarry@gmail.com>
@gen-xu gen-xu force-pushed the gen/fix_misaligned_dereference branch from d715918 to 74664c5 Compare July 25, 2023 09:53
@gen-xu
Copy link
Copy Markdown
Contributor Author

gen-xu commented Jul 25, 2023

@YangKeao just signed off

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants