Commit Graph

8 Commits

Author SHA1 Message Date
Anthony Latsis
9596dcd1df Gardening: Migrate test suite to GH issues: ClangImporter 2022-08-12 06:58:42 +03:00
Saleem Abdulrasool
0c63feb978 Revert "Move out enum tests into a separate CXXInterop/enum directory" 2020-04-15 09:13:19 -07:00
Rosica Dejanovska
396ba4f0e2 Move out enum tests into a separate CXXInterop/enum directory
Currently, tests are scattered throughout multiple directories, making
it difficult to determine what is tested and what is not tested.
With upcoming work on interoperability between Swift and C++
(https://forums.swift.org/t/manifesto-interoperability-between-swift-and-c/33874),
we'd like to rearrange the tests and bring tests covering a single C/C++
feature next to each other.
This PR also adds tests for enabling C++ interop where possible.
2020-03-02 16:32:55 +01:00
Saleem Abdulrasool
a54f57aad2 test: split up ClangImporter.enum-anon
This splits up the test into two halves in order to allow the test to
pass on Windows.  Loosen the test to allow the enumeration to be
imported with signed and unsigned types as Windows uses signed
enumerations by default.
2019-03-29 10:08:40 -07:00
Jordan Rose
e787906d65 [test] Update ClangImporter/enum-anon.swift for Clang improvements (#20804)
We previously weren't able to import something properly, but now it's
working because of some change in Clang. Let's take it!

rdar://problem/46263312
2019-02-08 14:22:56 -08:00
Erik Eckstein
39bb14b094 change mangling prefix from $S to $s
This is the final ABI mangling prefix

rdar://problem/38471478
2018-09-19 13:55:11 -07:00
Saleem Abdulrasool
7606e2b5c5 ClangImporter: improve ObjC-interop and enable tests
This enables additional tests for the ClangImporter.  This found a
missing piece in the `-enable-objc-interop` work that was done
previously.  Address that and enable the tests.  There are now the
following failing tests on Linux:

  * sdk - depends on Foundation (not hermetic, see SR-7572)
  * mixed-nsuinteger - depends on Foundation (not hermetic, see SR-7572)
  * import-mixed-with-header-twice - requires apple/swift PR#16022
  * can_import_objc_idempotent - requires apple/swift PR#16022
  * objc_protocol_renaming - requires apple/swift PR#16022
2018-05-01 18:39:42 -07:00
Jordan Rose
246a644723 [ClangImporter] Anonymous enums used as types can't be imported as Int (#13984)
Normally we treat anonymous enums as defining constants, and since we
don't know how those constants are going to be used we import them as
Int when we know we can do so consistently across all platforms. But
this isn't the right thing to do if the anonymous enum is being used
as an ad-hoc type for a variable or field.

(This may /still/ not be the right thing to do---maybe we want to
import it with a made-up name like we do anonymous fields instead.
But at least this way any such variables or fields become usable,
when they previously crashed IRGen.)

https://bugs.swift.org/browse/SR-2511
2018-01-23 16:09:39 -08:00