Skip to content

Commit

Permalink
[clang][test] zero-init-empty-virtual.cpp: remove redundant alignment…
Browse files Browse the repository at this point in the history
… check

Summary:
Fixes a `clang-armv8-quick` buildbot failure. Where the alignment
was `4` instead of `8`:
```
// CHECK: @{{.*}} = {{.*}}global %struct.Holder2 zeroinitializer, align 8
          ^
<stdin>:66:178: note: scanning from here
@g_holder1 = global %struct.Holder1 { %struct.polymorphic_base { ptr getelementptr inbounds inrange(-8, 12) ({ [5 x ptr] }, ptr @_ZTV16polymorphic_base, i32 0, i32 0, i32 2) } }, align 4
                                                                                                                                                                                 ^
<stdin>:68:8: note: possible intended match here
@g_holder2 = global %struct.Holder2 zeroinitializer, align 4
       ^
```

This test isn't about checking alignment, so remove that from the
FileCheck entry.

Test Plan: 

Reviewers: 

Subscribers: 

Tasks: 

Tags: 


Differential Revision: https://phabricator.intern.facebook.com/D60251625
  • Loading branch information
Michael137 authored and yuxuanchen1997 committed Jul 25, 2024
1 parent fff54b4 commit abd9962
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion clang/test/CodeGenCXX/zero-init-empty-virtual.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ struct Holder2 {
derived_virtual a{};
} g_holder2;

// CHECK: @{{.*}} = {{.*}}global %struct.Holder2 zeroinitializer, align 8
// CHECK: @{{.*}} = {{.*}}global %struct.Holder2 zeroinitializer

struct Holder3 {
derived a{};
Expand Down

0 comments on commit abd9962

Please sign in to comment.