[PATCH] D151059: [test] Add C++ ext_vector_type tests

This revision was automatically updated to reflect the committed changes. Closed by commit rG590eb76ba3cd: [test] Add C++ ext_vector_type tests (authored by porglezomp). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D151059/new/

[PATCH] D151059: [test] Add C++ ext_vector_type tests

fhahn accepted this revision. fhahn added a comment. This revision is now accepted and ready to land. LGTM, thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D151059/new/ https://reviews.llvm.org/D151059

[PATCH] D151059: [test] Add C++ ext_vector_type tests

porglezomp updated this revision to Diff 524429. porglezomp added a comment. Address review comments. File GitHub issues and reference them with the FIXMEs. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D151059/new/ https://reviews.llvm.org/D151059

[PATCH] D151059: [test] Add C++ ext_vector_type tests

aaron.ballman added a comment. Thank you for this coverage as well! Are there issues filed for the FIXME comments so we have a more prominent record of the problems? LGTM modulo comment concerns. Comment at: clang/test/SemaCXX/vector.cpp:554 +void

[PATCH] D151059: [test] Add C++ ext_vector_type tests

porglezomp added a comment. See also: C versions of these test cases: D151061 I think the C versions behave correctly and that C++ should match that behavior. Interested in feedback from anyone working with ext_vector_types. Repository: rG LLVM Github

[PATCH] D151059: [test] Add C++ ext_vector_type tests

porglezomp created this revision. porglezomp added reviewers: aaron.ballman, fhahn. Herald added a subscriber: StephenFan. Herald added a project: All. porglezomp requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Add initial tests for the