[Lldb-commits] [clang] [lldb] [UBSan][BoundsSafety] Implement support for more expressive "trap reasons" (PR #154618)

[Dan Liew via lldb-commits]

================
@@ -30,6 +30,7 @@ def CLASS_REMARK    : DiagClass;
 def CLASS_WARNING   : DiagClass;
 def CLASS_EXTENSION : DiagClass;
 def CLASS_ERROR     : DiagClass;
+def CLASS_TRAP      : DiagClass;
----------------
delcypher wrote:

My initial prototype didn't add a new class but after thinking about it I really didn't think any of the existing classes made much sense other than `error`. And as you say adding a class means we can enforce that we don't actually emit a trap diagnostic as a regular diagnostic.

I'll add the assert as you suggest.

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