llvmorg-github-actions[bot] wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-clang
Author: Hubert Tong (hubert-reinterpretcast)
<details>
<summary>Changes</summary>
Rename test file to reflect scope (includes array bounds checking).
Rename test functions for internal convention consistency:
Consistently use "deref" and "subscript" to refer to `*p` and `arr[idx]`
expression forms.
---
Full diff: https://github.com/llvm/llvm-project/pull/203731.diff
1 Files Affected:
- (renamed) clang/test/CodeGen/ubsan-aggregate-null-align-bounds.c (+4-4)
``````````diff
diff --git a/clang/test/CodeGen/ubsan-aggregate-null-align.c
b/clang/test/CodeGen/ubsan-aggregate-null-align-bounds.c
similarity index 96%
rename from clang/test/CodeGen/ubsan-aggregate-null-align.c
rename to clang/test/CodeGen/ubsan-aggregate-null-align-bounds.c
index 281a18ed5eb4c..9fc3fd6e64584 100644
--- a/clang/test/CodeGen/ubsan-aggregate-null-align.c
+++ b/clang/test/CodeGen/ubsan-aggregate-null-align-bounds.c
@@ -73,16 +73,16 @@ void test_rhs_ptrcheck_subscript(AGG arr[4]) {
// RHS cases - handler call only
-// CHECK-LABEL: define {{.*}}@test_init_from_ptr(
+// CHECK-LABEL: define {{.*}}@test_init_from_deref(
// CHECK: call void @__ubsan_handle_type_mismatch_v1_abort
-void test_init_from_ptr(AGG *src) {
+void test_init_from_deref(AGG *src) {
AGG local = *src;
(void)local;
}
-// CHECK-LABEL: define {{.*}}@test_init_from_array(
+// CHECK-LABEL: define {{.*}}@test_init_from_subscript(
// CHECK: call void @__ubsan_handle_type_mismatch_v1_abort
-void test_init_from_array(AGG arr[4]) {
+void test_init_from_subscript(AGG arr[4]) {
AGG local = arr[0];
(void)local;
}
``````````
</details>
https://github.com/llvm/llvm-project/pull/203731
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
