diff options
| author | David Blaikie <dblaikie@gmail.com> | 2021-10-14 14:07:51 -0700 |
|---|---|---|
| committer | David Blaikie <dblaikie@gmail.com> | 2021-10-14 14:23:32 -0700 |
| commit | 277623f4d5a672d707390e2c3eaf30a9eb4b075c (patch) | |
| tree | da210f657ede3dbbed000adbb6f581757c8ab132 /clang/test/Layout/ms-x86-basic-layout.cpp | |
| parent | aeeefe97c686dd51e2614d2a3e4214701c57371b (diff) | |
Compress formatting of array type names (int [4] -> int[4])
Based on post-commit review discussion on
2bd84938470bf2e337801faafb8a67710f46429d with Richard Smith.
Other uses of forcing HasEmptyPlaceHolder to false seem OK to me -
they're all around pointer/reference types where the pointer/reference
token will appear at the rightmost side of the left side of the type
name, so they make nested types (eg: the "int" in "int *") behave as
though there is a non-empty placeholder (because the "*" is essentially
the placeholder as far as the "int" is concerned).
Diffstat (limited to 'clang/test/Layout/ms-x86-basic-layout.cpp')
| -rw-r--r-- | clang/test/Layout/ms-x86-basic-layout.cpp | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/clang/test/Layout/ms-x86-basic-layout.cpp b/clang/test/Layout/ms-x86-basic-layout.cpp index 0b49fcb0b074..1bd85bf6057e 100644 --- a/clang/test/Layout/ms-x86-basic-layout.cpp +++ b/clang/test/Layout/ms-x86-basic-layout.cpp @@ -716,11 +716,11 @@ struct ArrayFieldOfRecords { }; // CHECK-LABEL: 0 | struct ArrayFieldOfRecords{{$}} -// CHECK-NEXT: 0 | struct A4 [2] InlineElts +// CHECK-NEXT: 0 | struct A4[2] InlineElts // CHECK-NEXT: | [sizeof=8, align=4 // CHECK-NEXT: | nvsize=8, nvalign=4] // CHECK-X64-LABEL: 0 | struct ArrayFieldOfRecords{{$}} -// CHECK-X64-NEXT: 0 | struct A4 [2] InlineElts +// CHECK-X64-NEXT: 0 | struct A4[2] InlineElts // CHECK-X64-NEXT: | [sizeof=8, align=4 // CHECK-X64-NEXT: | nvsize=8, nvalign=4] @@ -729,11 +729,11 @@ struct ArrayOfArrayFieldOfRecords { }; // CHECK-LABEL: 0 | struct ArrayOfArrayFieldOfRecords{{$}} -// CHECK-NEXT: 0 | struct A4 [2][2] InlineElts +// CHECK-NEXT: 0 | struct A4[2][2] InlineElts // CHECK-NEXT: | [sizeof=16, align=4 // CHECK-NEXT: | nvsize=16, nvalign=4] // CHECK-X64-LABEL: 0 | struct ArrayOfArrayFieldOfRecords{{$}} -// CHECK-X64-NEXT: 0 | struct A4 [2][2] InlineElts +// CHECK-X64-NEXT: 0 | struct A4[2][2] InlineElts // CHECK-X64-NEXT: | [sizeof=16, align=4 // CHECK-X64-NEXT: | nvsize=16, nvalign=4] @@ -743,11 +743,11 @@ struct RecordArrayTypedef { }; // CHECK-LABEL: 0 | struct RecordArrayTypedef{{$}} -// CHECK-NEXT: 0 | RecordArrayTypedef::ArrayTy [2] InlineElts +// CHECK-NEXT: 0 | RecordArrayTypedef::ArrayTy[2] InlineElts // CHECK-NEXT: | [sizeof=16, align=4 // CHECK-NEXT: | nvsize=16, nvalign=4] // CHECK-X64-LABEL: 0 | struct RecordArrayTypedef{{$}} -// CHECK-X64-NEXT: 0 | RecordArrayTypedef::ArrayTy [2] InlineElts +// CHECK-X64-NEXT: 0 | RecordArrayTypedef::ArrayTy[2] InlineElts // CHECK-X64-NEXT: | [sizeof=16, align=4 // CHECK-X64-NEXT: | nvsize=16, nvalign=4] @@ -756,11 +756,11 @@ struct EmptyIntMemb { }; // CHECK-LABEL: 0 | struct EmptyIntMemb{{$}} -// CHECK-NEXT: 0 | int [0] FlexArrayMemb +// CHECK-NEXT: 0 | int[0] FlexArrayMemb // CHECK-NEXT: | [sizeof=1, align=4 // CHECK-NEXT: | nvsize=0, nvalign=4] // CHECK-X64-LABEL: 0 | struct EmptyIntMemb{{$}} -// CHECK-X64-NEXT: 0 | int [0] FlexArrayMemb +// CHECK-X64-NEXT: 0 | int[0] FlexArrayMemb // CHECK-X64-NEXT: | [sizeof=4, align=4 // CHECK-X64-NEXT: | nvsize=0, nvalign=4] @@ -769,11 +769,11 @@ struct EmptyLongLongMemb { }; // CHECK-LABEL: 0 | struct EmptyLongLongMemb{{$}} -// CHECK-NEXT: 0 | long long [0] FlexArrayMemb +// CHECK-NEXT: 0 | long long[0] FlexArrayMemb // CHECK-NEXT: | [sizeof=1, align=8 // CHECK-NEXT: | nvsize=0, nvalign=8] // CHECK-X64-LABEL: 0 | struct EmptyLongLongMemb{{$}} -// CHECK-X64-NEXT: 0 | long long [0] FlexArrayMemb +// CHECK-X64-NEXT: 0 | long long[0] FlexArrayMemb // CHECK-X64-NEXT: | [sizeof=8, align=8 // CHECK-X64-NEXT: | nvsize=0, nvalign=8] |