Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[SR-6387] Build unittests with the just-built clang #48937

Open
gparker42 mannequin opened this issue Nov 15, 2017 · 2 comments
Open

[SR-6387] Build unittests with the just-built clang #48937

gparker42 mannequin opened this issue Nov 15, 2017 · 2 comments
Labels
bug A deviation from expected or documented behavior. Also: expected but undesirable behavior. compiler The Swift compiler in itself

Comments

@gparker42
Copy link
Mannequin

gparker42 mannequin commented Nov 15, 2017

Previous ID SR-6387
Radar rdar://problem/35564415
Original Reporter @gparker42
Type Bug
Additional Detail from JIRA
Votes 0
Component/s Compiler
Labels Bug
Assignee Rostepher (JIRA)
Priority Medium

md5: 3ef680a5fa7ec45c9ab36de291bab74b

Issue Description:

The tests in swift/unittests are built with the system clang. They should use Swift's locally-built clang instead.

One problem is that on Linux the system clang may ignore the swiftcall attribute. Code built without that attribute may be ABI-incompatible with other Swift code.

This problem surfaces in the Linux long-test jobs on ci.swift.org:
@belkadan
Copy link
Contributor

cc Rostepher (JIRA User), currently-designated Keeper of the CMake.

@swift-ci
Copy link
Collaborator

Comment by Ross Bayer (JIRA)

@swift-ci create

@swift-ci swift-ci transferred this issue from apple/swift-issues Apr 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug A deviation from expected or documented behavior. Also: expected but undesirable behavior. compiler The Swift compiler in itself
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@belkadan @swift-ci