llvmbot wrote:

<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang

Author: Orlando Cazalet-Hyams (OCHyams)

<details>
<summary>Changes</summary>

It doesn't need to be a Key Instruction.

(I noticed I had this test lying around on one of my branches - the extra 
coverage can't hurt?)

---
Full diff: https://github.com/llvm/llvm-project/pull/146517.diff


1 Files Affected:

- (added) clang/test/DebugInfo/KeyInstructions/array-cookie.cpp (+12) 


``````````diff
diff --git a/clang/test/DebugInfo/KeyInstructions/array-cookie.cpp 
b/clang/test/DebugInfo/KeyInstructions/array-cookie.cpp
new file mode 100644
index 0000000000000..cfa343551d162
--- /dev/null
+++ b/clang/test/DebugInfo/KeyInstructions/array-cookie.cpp
@@ -0,0 +1,12 @@
+// RUN: %clang_cc1 -triple x86_64-linux-gnu -gkey-instructions %s 
-debug-info-kind=line-tables-only -gno-column-info -emit-llvm -o - \
+// RUN: | FileCheck %s
+
+// Array cookie store doesn't need to be a key instruction.
+
+struct a { char c; ~a(); };
+void b() { new a[2]; }
+
+// CHECK:      %call = call {{.*}}ptr @_Znam(i64 noundef 10)
+// CHECK-NEXT: store i64 2, ptr %call, align 8, !dbg [[DBG:!.*]]
+
+// CHECK: [[DBG]] = !DILocation(line: 7, scope: ![[#]])

``````````

</details>


https://github.com/llvm/llvm-project/pull/146517
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to