Skip to content

verify: Send crash reports to cpp-verify's issue tracker - #46

Merged
SwayamInSync merged 4 commits into
mainfrom
fix/bug-report-url
Oct 22, 2026
Merged

SwayamInSync merged 4 commits into
mainfrom
fix/bug-report-url

Conversation

@SwayamInSync

Copy link
Copy Markdown
Owner

A crash of cpp-verify, or of the clang it is built with, asked for a report on LLVM's issue tracker:

PLEASE submit a bug report to https://github.com/llvm/llvm-project/issues/ and include the crash backtrace and instructions to reproduce the bug.

Those reports belong to this repository. PACKAGE_BUGREPORT in llvm/CMakeLists.txt, the default of the BUG_REPORT_URL cache variable that both llvm/Config/config.h and clang/Config/config.h take, now names https://github.com/SwayamInSync/cpp-verify/issues/. LLVM's stack-trace handler and the Clang driver both print it.

A build directory configured before this keeps LLVM's address in its CMake cache, because BUG_REPORT_URL is a cache variable; cmake -UBUG_REPORT_URL <build> picks up the new default. CI and release builds configure fresh.

Docs: the tooling page gains a section on what a crash prints, what to include in a report, and the stale cache. The release notes get an entry under "Running cpp-verify".

Testing: Verify/crash_report_url.cpp crashes cpp-verify and clang -cc1 with #pragma clang __debug crash and checks the address. It uses not --crash and REQUIRES: crash-recovery, backtrace, as Clang's crash tests do; not --crash already treats an abort on Windows as a crash. Both lines fail on main's binary. clang++ from the same build prints the address too. Clang's crash-report tests, which match the address with {{.*}}, still pass (Driver/crash-report*.{test,m,cpp}, Driver/crash-diagnostics-tar.c, Analysis/crash-trace.c; crash-report-crashfile.m is Darwin-only). The documentation examples pass and the site builds with -W.

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.

1 participant