[clang] [ubsan] Add more -fsanitize-annotate-debug-info checks (PR #141997)

Thurston Dang via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 2 15:58:43 PDT 2025


================
@@ -2756,10 +2758,23 @@ CodeGenFunction::SanitizerScope::SanitizerScope(CodeGenFunction *CGF)
     : CGF(CGF) {
   assert(!CGF->IsSanitizerScope);
   CGF->IsSanitizerScope = true;
+
+  assert(!this->ApplyTrapDI);
+}
+
+CodeGenFunction::SanitizerScope::SanitizerScope(
+    CodeGenFunction *CGF, ArrayRef<SanitizerKind::SanitizerOrdinal> Ordinals,
+    SanitizerHandler Handler)
+    : SanitizerScope(CGF) {
+  this->ApplyTrapDI = new ApplyDebugLocation(
----------------
thurstond wrote:

> std::make_unuque<>

Done

> But why do we need heap allocation here?

ApplyDebugLocation is scoped.

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


More information about the cfe-commits mailing list