This is an archive of the discontinued LLVM Phabricator instance.

Differential D50550

[ASTImporter] Added test case for opaque enums
ClosedPublic

Authored by teemperor on Aug 9 2018, 10:04 PM.

Details

Reviewers
a.sidorin
a_sidorin
Commits
rG612c524a9047: [ASTImporter] Added test case for opaque enums
rC339506: [ASTImporter] Added test case for opaque enums
rL339506: [ASTImporter] Added test case for opaque enums

Diff Detail

Event Timeline

teemperor created this revision.Aug 9 2018, 10:04 PM
Herald added a reviewer: a.sidorin. · View Herald TranscriptAug 9 2018, 10:04 PM
Herald added a subscriber: martong. · View Herald Transcript
teemperor updated this revision to Diff 160057.Aug 9 2018, 10:22 PM
  • Removed broken FileCheck directive
a_sidorin accepted this revision.Aug 11 2018, 1:32 PM
a_sidorin added a subscriber: a_sidorin.

LGTM!

This revision is now accepted and ready to land.Aug 11 2018, 1:32 PM
Closed by commit rL339506: [ASTImporter] Added test case for opaque enums (authored by teemperor). · Explain WhyAug 11 2018, 4:44 PM
This revision was automatically updated to reflect the committed changes.
Herald added a subscriber: llvm-commits. · View Herald TranscriptAug 11 2018, 4:44 PM

Revision Contents

PathSize
test/
Import/
enum/
Inputs/
2 lines
7 lines

Diff 160056

test/Import/enum/Inputs/S.cpp

enum E { enum E {
a = 1, a = 1,
b = 2 b = 2
}; };
enum OpaqueWithType : long;

test/Import/enum/test.cpp

// RUN: clang-import-test -import %S/Inputs/S.cpp -expression %s // RUN: clang-import-test -dump-ast -import %S/Inputs/S.cpp -expression %s | FileCheck %s
// CHECK: OpaqueWithType 'long'
// CHECK-SANE: EnumDecl
void expr() { void expr() {
static_assert(E::a + E::b == 3); static_assert(E::a + E::b == 3);
static_assert(sizeof(OpaqueWithType) == sizeof(long));
} }