[clang] [llvm] [clang][LLVM Demangler] Add an assertion that validates that all mang… (PR #111391)

Viktoriia Bakalova via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 8 01:19:25 PDT 2024


VitaNuo wrote:

> Since this is being done as an assert, this is something that really only applies during debug mode. I dont think this compiler flag makes sense because of it.

The reason I am adding a flag on top of an assert is that there are unfortunately still many mangled names produced by `clang` that cannot be demangled. E.g., many of the existing codegen lit tests (`clang/test/CodeGenCXX/*`) fail in assertion-enabled builds without the flag. This is potentially disruptive to development workflows. The flag can be removed once the LLVM demangler can handle all the mangled names `clang` produces.

https://github.com/llvm/llvm-project/pull/111391


More information about the cfe-commits mailing list