Binary mode carries the previous function's source line into functions with no line info
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 79/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- cpp, typescript
Hướng nghiên cứu
Bắt đầu bằng cách đọc phần xử lý nhãn hàm trong bộ phân tích cú pháp objdump ở chế độ nhị phân và so sánh với thay đổi TypeScript trong compiler-explorer/compiler-explorer#9306. Kiểm tra cách các nhãn .L và nhãn hàm xuất hiện trong các ví dụ dump thực tế được cung cấp hoặc trong các bài kiểm thử của bộ phân tích cú pháp. Hoàn tất khi source hiện tại được xóa tại các nhãn hàm không phải .L, source được giữ nguyên qua các nhãn .L và xác nhận rằng các ánh xạ không còn rò rỉ vào những hàm không có bản ghi dòng.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
In binary mode ("Link to binary"), a function with no line records of its own, such as statically linked glibc code built without debug info, inherits the last file:line of the function before it. objdump only prints a line record when the line changes, and the parser doesn't clear the current source at a new function label.
Reproduce on godbolt.org with x86-64 gcc 14.2 (cg142), C, with binary output on and the default filters:
int square(int num) {
return num * num;
}
int main() { return square(3); }
- With
-O1 -g, everything is correct: 5 lines map toexample.c, all insquareandmain. - With
-O1 -g -static, 4688 of the 5001 lines map toexample.c, across 76 glibc functions:init_cpu_features.constprop.0810,update_active.constprop.0326,get_common_cache_info.constprop.0321, and so on. Onlysquareandmainshould.
With a normal dynamic link it's hidden, because binaryHideFuncRe hides the CRT functions.
Fix: clear the current source at each function label, except assembler-local .L labels, which are inside a function (RISC-V keeps .Lpcrel_hi* in the symbol table). objdump -l prints a function's first line record after its label, so nothing correct is lost. CE's TypeScript parser got exactly this in compiler-explorer/compiler-explorer#9306:
func = match[2];
if (!func.startsWith('.L')) source = null;
That PR has the details and tests. The reviewer checked it against 24 real dumps (gcc and clang, -O0/-O2, with and without -g, -static, C and C++), and the mappings matched addr2line. Linux prod C/C++ uses this external parser (externalparser=CEAsmParser), so the TypeScript fix doesn't reach it.
🤖 Generated with Claude Code
- Ngôn ngữ chính
- C++
- Star
- 36
- Fork
- 10
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Chuẩn bị môi trường
Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
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.
Issue khác của compiler-explorer/asm-parser
-
Add aarch64 buildĐang mở
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 20/100
-
Make enum out of the parser states (objdump/parser)Có thể làm lại được @phyBrackets đã nhận 1726 ngày trước và không có pull request nào đang mở. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 55/100
compiler-explorer/asm-parser#21 · 2 bình luận ·
Tất cả issue của compiler-explorer/asm-parser
Issue tương tự
-
Add string concatenationĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Failed-capture notification says "Save diagnostics for this upload" when nothing was uploadedĐang mởarea: ui bug good first issue P2: medium value: medium
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
patrick-hudson/put.gd-desktop#366 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
Niko1221/Strata#3673 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 3 ngày