WebAssembly / WebAssembly/binaryen
Change default inlining behavior to favor single-caller inlines
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- WebAssembly
- Star
- 8.6k
- Fork
- 885
- Merge trung bình
- 1 ngày 19 giờ
- Pull request đã merge (30 ngày)
- 69
Mô tả
See FIXMEs introduced in: https://github.com/WebAssembly/binaryen/pull/2125
Essentially, the check for calls == 1 need to be moved first in that function, otherwise oneCallerInlineMaxSize can never be bigger than flexibleInlineMaxSize, which naturally it would make sense for it to be.
@kripken mentioned this change can only be made after it has been checked if it doesn't regress size/speed.
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Hướng nghiên cứu
Bắt đầu với các thay đổi FIXME được giới thiệu bởi pull request 2125 và tìm hàm inlining chứa kiểm tra calls == 1. Hãy xác minh thay đổi về thứ tự bằng các phép đo kích thước và tốc độ trước khi coi công việc là hoàn tất; issue không cung cấp tệp kiểm thử hoặc benchmark cụ thể nào.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- cpp, wasm
- Lĩnh vực
- compilers, performance
- Loại issue
- Tính năng
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 35/100