llvmorg-github-actions[bot] wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang Author: mkovacevic99 <details> <summary>Changes</summary> Follow-up to #<!-- -->197874. The codegen test was missing FileCheck lines, making it a no-op. Add checks that verify both __auto_type _Atomic and _Atomic __auto_type produce correct allocas and stores. Follow up to comment from: https://github.com/llvm/llvm-project/pull/197874#discussion_r3496368126 --- Full diff: https://github.com/llvm/llvm-project/pull/206749.diff 1 Files Affected: - (modified) clang/test/CodeGen/atomic-auto-type.c (+7-1) ``````````diff diff --git a/clang/test/CodeGen/atomic-auto-type.c b/clang/test/CodeGen/atomic-auto-type.c index 0969f22596421..dd9f587b92b71 100644 --- a/clang/test/CodeGen/atomic-auto-type.c +++ b/clang/test/CodeGen/atomic-auto-type.c @@ -1,9 +1,15 @@ -// RUN: %clang_cc1 -triple x86_64-pc-linux-gnu -emit-llvm -o - %s > /dev/null +// RUN: %clang_cc1 -triple x86_64-pc-linux-gnu -emit-llvm -o - %s | FileCheck %s // This is a regression test for handling of __auto_type inside _Atomic. // Previously this could lead to an undeduced AutoType escaping into // ASTContext::getTypeInfoImpl and causing an assertion failure. +// CHECK-LABEL: define {{.*}} @f( +// CHECK: %xa = alloca double +// CHECK: %ax = alloca double +// CHECK: store double {{.*}}, ptr %xa +// CHECK: store double {{.*}}, ptr %ax + void f(double x) { __auto_type _Atomic xa = x; _Atomic __auto_type ax = x; `````````` </details> https://github.com/llvm/llvm-project/pull/206749 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
