Skip to content

Conversation

@waskyo
Copy link
Contributor

@waskyo waskyo commented Apr 28, 2025

the prototype doesn't match what one of the binaries we're analyzing uses (it doesn't have the last 'fun' parameter).

Hopefully this doesn't cause issues regarding the non-returning semantics and the other api attributes

the prototype doesn't match what one of the binaries we're
analyzing uses (it doesn't have the last 'fun' parameter).

Hopefully this doesn't cause issues regarding the non-returning
semantics and the other api attributes
Copy link
Contributor

@sipma sipma 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!

Removing this function summary may indeed affect function construction. I have raised a new issue to create the option to address this via gcc-attributes in the header (#191).

@sipma sipma merged commit c443f5e into static-analysis-engineering:master Apr 28, 2025
24 checks passed
@waskyo waskyo deleted the remove_assert_fail_summary branch April 29, 2025 16:10
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.

2 participants