[clang] Add support for builtin_verbose_trap (PR #79230)

Dan Liew via cfe-commits cfe-commits at lists.llvm.org
Sat Apr 27 00:24:53 PDT 2024


================
@@ -27,6 +27,9 @@ namespace llvm {
   }
 }
 
+// Prefix for __builtin_verbose_trap.
+#define CLANG_VERBOSE_TRAP_PREFIX "__llvm_verbose_trap"
----------------
delcypher wrote:

@ahatanak Nit: Why does the macro name mention `CLANG` but the string mentions `llvm` instead of Clang? Seems like this prefix is specific to Clang and not LLVM.

Also here’s suggestion for a shorter prefix `__llvm_trap_msg`. It’s not a big deal so feel to ignore this advice. 

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


More information about the cfe-commits mailing list