Builds without a build type are unoptimized: the README's commands and CI's main job
Maintainer thường phản hồi trong vòng 1 ngày
@SizzleUnrlsd đang làm issue này rồi.
Từ ngày 15/10/2026.
Đánh giá
Issue này chưa được đánh giá.
Mô tả
Neither CMakeLists.txt nor the stack analyzer sets a default CMAKE_BUILD_TYPE. With a single-config generator such as Ninja, an empty build type compiles without optimization: no -O flag. Two builds are affected:
- the README's BUILD commands, for Ubuntu 24.04 and macOS;
- CI's main job (
build.yml, "Build on ubuntu-24.04 / macos-14"). Itscmake --build build --config Releasehas no effect, because Ninja ignores--config.
The release archives (Dockerfile.release-binaries) and the coverage job pass -DCMAKE_BUILD_TYPE=Release.
Observed
ctrace --compile-commands <CoreTrace's 22 translation units> --invoke ctrace_stack_analyzer, on Linux x86_64 (32 cores) with LLVM 20.1.2, a fresh cache, and one run at a time:
| Stack analyzer | No build type | Release |
|---|---|---|
| v0.22.0 | 29 min 39 s | 3 min 16 s |
main 0668fb8 |
5 min 18 s | 1 min 33 s |
The run of #169 that was still going after 42 minutes was an unoptimized build too.
To do
- When
CMAKE_BUILD_TYPEis empty and the generator is single-config, default it toRelease. - Remove the
--config Releasethat does nothing from the main CI job. - Decide which build type CI's main job tests. Today it builds the analyzers that
ctracelinks with their assertions enabled (noNDEBUG), and aReleasedefault would turn them off. To keep them, the job can pass its build type explicitly.
Acceptance
- The README's commands build an optimized
ctrace. - The main CI job states the build type it tests.
- Ngôn ngữ chính
- C++
- Star
- 5
- Fork
- 4
- Merge trung bình
- 1 giờ 57 phút
- Pull request đã merge (30 ngày)
- 88
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc 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.
Issue khác của CoreTrace/coretrace
-
chore good first issue severity/minor
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
Maintainer thường phản hồi trong vòng 1 ngày
-
build severity/minor
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
CoreTrace/coretrace#176 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
build good first issue severity/minor
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Inputs: a file name starting with @ may be read as a response file by the compilerCó thể đã có người làm @SizzleUnrlsd đã nhận 2 ngày trước. Đang mởbug severity/minor
CoreTrace/coretrace#203 · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug severity/minor
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 50/100
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của CoreTrace/coretrace
Issue tương tự
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 91/100
FujiNetWIFI/fujinet-firmware#2045 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
(S1 - Need confirmation)
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
CleverRaven/Cataclysm-DDA#89159 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
area:libnvmpi bug P3
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
gjrtimmer/jetson-nvmpi#33 ·
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 74/100
Maintainer thường phản hồi trong vòng 1 ngày
-
bug(wasi_crypto): ArrayOutput::pull reports the wrong all-pulled flagCó thể đã có người làm @Yashbhu đã nhận hôm nay. Đang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 84/100
Maintainer thường phản hồi trong vòng 1 ngày