================
@@ -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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to