Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Support musl #16

Open
wants to merge 18 commits into
base: master
Choose a base branch
from
Open

Support musl #16

wants to merge 18 commits into from

Conversation

joii2020
Copy link

@joii2020 joii2020 commented Dec 3, 2024 •
edited
Loading

Changes in cycles and binary size.
Cycles: tests/test_cases/msgpack-benchmark.lua
Binary size : lua-loader

Cycles binary size
Old GCC 7431924(7.1M) 238328(233K)
Old LLVM 7406131(7.1M) 211232(206K)
New GCC 7428954(7.1M) 238328(233K)
New LLVM 8376910(8.0M) 259600(254K)

@joii2020 joii2020 changed the title [WIP] Support musl Support musl Dec 9, 2024
@XuJiandong
Copy link
Collaborator

This PR will not be merged for now, based on the following reasons:

  1. The code has become more confusing, with both musl and my_xxx.
  2. The compiler uses both gcc and clang.
  3. The binary size has increased by approximately 50KB.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@joii2020 @XuJiandong